Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The database index "idxVulnerability" on the column "vulnerability.cve" is redundant #6788

Closed
nikita-efremov1 opened this issue Jul 3, 2024 · 1 comment
Closed

The database index "idxVulnerability" on the column "vulnerability.cve" is redundant #6788

nikita-efremov1 opened this issue Jul 3, 2024 · 1 comment
Labels
question

Comments

@nikita-efremov1
Copy link
Contributor

nikita-efremov1 commented Jul 3, 2024 •
edited
Loading

We have found two indices on the "vulnerability.cve" column in the postgreSql database:

  1. The first one is created automatically because of column uniqueness.
  2. The second one is created manually.

Since "cve" column is unique is doesn't make sense to create a separated index.
Sometimes it might lead to performance issues.
This is relevant at least for postgreSql, Oracle, mysql and mssql databases.

Screenshot 2024-07-02 at 13 58 34
@jeremylong
Copy link
Owner

good enough - the second index can be removed. We accept PRs.

nikita-efremov1 added a commit to nikita-efremov1/DependencyCheck that referenced this issue Jul 5, 2024
@nikita-efremov1
fix(db): jeremylong#6788 removing redundant db index "idxVulnerabilit…
e0ba5d3 
…y" on "vulnerability.cve"
@nikita-efremov1 nikita-efremov1 mentioned this issue Jul 5, 2024
Merged
jeremylong pushed a commit that referenced this issue Jul 5, 2024
@nikita-efremov1
fix(db): #6788 removing redundant db index "idxVulnerability" on "vul…
06e39fc 
…nerability.cve" (#6807)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jeremylong @nikita-efremov1