feat(query_perf_monitoring): Implement query performance monitoring metrics #191
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rahulreddy15
approved these changes
Jan 23, 2025
Query performance monitoring undo change remove test query query performance monitoring undo changelog changes Query performance monitoring Query performance monitoring slow query len check in individual queries metrics populate trim query text lint issue fix undo changelog file change test remove OH queries remove OH queries reusing existing DB connection (#22) reusing existing DB connection DB specific metric collection (#23) DB specific metric collection query response time threshold (#24) query response time threshold change in individual query (#25) change in individual query version specific queries (#26) verison specific queries fix database name (#27) fix database name test test min max threshold on limit (#28) min max threshold on limit Feat version specific individual query (#29) * version specific indivual query Feat version specific individual query (#30) Version specific indivual query fix query id , ingestData:false and reuse individual version specific query (#31) fix query id , ingestData:false and reuse individual version specific query database list check (#32) database list check publish only if there are metrics (#33) publish only if there are metrics undo change log change
Refactor : version specific blocking session annonamization
refactor: resolved review comments Co-authored-by: jsurampudi <[email protected]>
* refactor: resolved review comments and lint issues Co-authored-by: tharun0064 <[email protected]>
Refactor : nil checks
resolved review comments
* resolved: review comments
* Integration Testing Setup -- Needs Cleanup * fixed old integration test * tests now check unsupported containers * cleanup and fixed linting errors
feat : resolved review comments
resolved: review comments
* resolved: review comments
Feat: resolved review comments
* Refactored Integration Tests * Fixed linting errors * Fixed linting errors again * Fixed linting errors again again * Fixed linting errors again again again * linting is back * linting is back again * linting is back again again * linting is back again again again * linting is back again again again again
resolved : review comments
fix lint issues
Feat : resolved review comments
* resolved: refactored unit test cases
rahulreddy15
force-pushed
the
feat_performance_monitoring
branch
from
090797b to
e7dff9d
Compare
sairaj18
merged commit b243aa1
into
newrelic:pre-release_query_performance_monitoring
18 of 19 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Creating this PR to unblock tara-team form creating pre-releases of the integration with the changes of query performance monitoring.
Going forward the pre-release_query_performance branch in the nri-postgresql main repo will be used to create new pre-releases until we merge PRs related to forked repo epic to main repo epic