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.

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

Java: add diagnostic query indicating low database quality #16810

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

smowton
Copy link
Contributor

@smowton smowton commented Jun 21, 2024

No description provided.

@smowton smowton requested a review from a team as a code owner June 21, 2024 16:08
@github-actions github-actions bot added the Java label Jun 21, 2024
@smowton smowton requested a review from henrymercer June 21, 2024 16:09
@saritai saritai added the ready-for-doc-review This PR requires and is ready for review from the GitHub docs team. label Jun 24, 2024
@smowton smowton force-pushed the smowton/feature/java-low-db-quality-query branch from db4ea46 to 884bd20 Compare June 24, 2024 15:38
saritai
saritai previously approved these changes Jun 24, 2024
Copy link
Contributor

@saritai saritai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@smowton Some small style things in the warning text but overall looks good from a docs perspective!

henrymercer
henrymercer previously approved these changes Jun 25, 2024
java/ql/src/Telemetry/DatabaseQualityDiagnostics.ql Outdated Show resolved Hide resolved
@smowton smowton dismissed stale reviews from henrymercer and saritai via 40dcca5 June 25, 2024 14:59
@smowton smowton force-pushed the smowton/feature/java-low-db-quality-query branch from ad34a58 to 51c21b7 Compare June 25, 2024 15:14
@smowton smowton requested a review from a team as a code owner June 25, 2024 15:50
Copy link
Contributor

@aeisenberg aeisenberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes a lot of sense and will be really useful. I do have a question about the change in the workflows.

.github/workflows/compile-queries.yml Outdated Show resolved Hide resolved
@aeisenberg
Copy link
Contributor

Also, does this affect C# as well? Should we make the same change for that language, too?

@coadaflorin
Copy link
Contributor

Also, does this affect C# as well? Should we make the same change for that language, too?

It does, but we just shipped to public beta to C# and we want to wait a little while to look at the data to see where we put the line for 🟡 / 🟢

@yo-h
Copy link
Contributor

yo-h commented Jun 26, 2024

Green is meant to indicate that the quality of the database is close to a traced database. Yellow is meant to indicate that it is not. For both Java and C#, 95% seems to be a sensible threshold based on that principle. This may result in more % of databases being flagged for C#, but that makes sense, as it's consistent with the data.

@smowton smowton force-pushed the smowton/feature/java-low-db-quality-query branch from d8e7874 to 9be2c1b Compare June 26, 2024 15:49
smowton and others added 7 commits June 27, 2024 09:57
Co-authored-by: Sarita Iyer <66540150+saritai@users.noreply.github.com>
Co-authored-by: Chad Bentz <1760475+felickz@users.noreply.github.com>
@smowton smowton force-pushed the smowton/feature/java-low-db-quality-query branch from 9be2c1b to 5cee392 Compare June 27, 2024 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Java ready-for-doc-review This PR requires and is ready for review from the GitHub docs team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants