Feat review comments #63
Merged
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
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
fix lint issues
Feat : resolved review comments
rahulreddy15
added a commit
that referenced
this pull request
Feb 12, 2025
* remove unused variables --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]>
tharun0064
added a commit
that referenced
this pull request
Feb 13, 2025
* Query performance monitoring 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 (#35) Refactor : version specific blocking session annonamization * refactor: resolved review comments (#36) * Feat review comments (#37) refactor: resolved review comments Co-authored-by: jsurampudi <[email protected]> * Feat review comments (#39) * refactor: resolved review comments and lint issues Co-authored-by: tharun0064 <[email protected]> * compile issue - resolved * compile issue - resolved * refactor : rename variable * Refactor : nil checks (#40) Refactor : nil checks * Feat : resolved review comments (#42) resolved review comments * resolved: review comments (#44) * resolved: review comments * Integration testing (#47) * Integration Testing Setup -- Needs Cleanup * fixed old integration test * tests now check unsupported containers * cleanup and fixed linting errors * Feat : resolved review comments (#48) feat : resolved review comments * Feat : resolved review comments (#49) resolved: review comments * Feat review comments (#51) * resolved: review comments * Feat: resolved review comments (#52) Feat: resolved review comments * Refactored Integration Tests (#53) * 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 * Feat review comments (#54) resolved : review comments * fix lint issues (#55) fix lint issues * Feat : resolved review comments (#56) Feat : resolved review comments * Feat: refactored unit test cases (#57) * resolved: refactored unit test cases * Upgrade Go Version * Feat: resolved review comments (#58) * resolved : review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat review comments (#59) fix: lint issues --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat: resolved review comments (#60) * added more scenerios for unit test cases --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat resolved review comments (#61) * rename variable --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat : resolved review comments (#62) resolved : review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat review comments (#63) * remove unused variables --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat : resolved review comments (#64) * resolved: review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat: resolved review comments (#65) Feat: resolved review comments * Feat : resolved review comments (#66) * add check while fetching execution plan * resolved : lint issues --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * remove has metrics (#69) * Feat review comments (#67) Fix: Resolved review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * resolved: review comments (#70) fix: lint issues * Execution plan datamodal fix (#71) * execution plan data model fix * Add default description (#72) * add default description --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat review comments (#73) * feat: resolved review comments * update execution plan schema * feat: change in variable name * feat: change in variable --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Oldest supprted postgres version is version 13 * Feat : resolved review comments (#74) * feat: resolved review commets * fix: lint issues --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Remove unnecasary list command --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]>
tharun0064
added a commit
that referenced
this pull request
Feb 13, 2025
* Query performance monitoring 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 (#35) Refactor : version specific blocking session annonamization * refactor: resolved review comments (#36) * Feat review comments (#37) refactor: resolved review comments Co-authored-by: jsurampudi <[email protected]> * Feat review comments (#39) * refactor: resolved review comments and lint issues Co-authored-by: tharun0064 <[email protected]> * compile issue - resolved * compile issue - resolved * refactor : rename variable * Refactor : nil checks (#40) Refactor : nil checks * Feat : resolved review comments (#42) resolved review comments * resolved: review comments (#44) * resolved: review comments * Integration testing (#47) * Integration Testing Setup -- Needs Cleanup * fixed old integration test * tests now check unsupported containers * cleanup and fixed linting errors * Feat : resolved review comments (#48) feat : resolved review comments * Feat : resolved review comments (#49) resolved: review comments * Feat review comments (#51) * resolved: review comments * Feat: resolved review comments (#52) Feat: resolved review comments * Refactored Integration Tests (#53) * 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 * Feat review comments (#54) resolved : review comments * fix lint issues (#55) fix lint issues * Feat : resolved review comments (#56) Feat : resolved review comments * Feat: refactored unit test cases (#57) * resolved: refactored unit test cases * Upgrade Go Version * Feat: resolved review comments (#58) * resolved : review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat review comments (#59) fix: lint issues --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat: resolved review comments (#60) * added more scenerios for unit test cases --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat resolved review comments (#61) * rename variable --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat : resolved review comments (#62) resolved : review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat review comments (#63) * remove unused variables --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat : resolved review comments (#64) * resolved: review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat: resolved review comments (#65) Feat: resolved review comments * Feat : resolved review comments (#66) * add check while fetching execution plan * resolved : lint issues --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * remove has metrics (#69) * Feat review comments (#67) Fix: Resolved review comments --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * resolved: review comments (#70) fix: lint issues * Execution plan datamodal fix (#71) * execution plan data model fix * Add default description (#72) * add default description --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Feat review comments (#73) * feat: resolved review comments * update execution plan schema * feat: change in variable name * feat: change in variable --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Oldest supprted postgres version is version 13 * Feat : resolved review comments (#74) * feat: resolved review commets * fix: lint issues --------- Co-authored-by: Jyothi Surampudi <[email protected]> Co-authored-by: jsurampudi <[email protected]> Co-authored-by: Rahul Reddy <[email protected]> * Remove unnecasary list command --------- Co-authored-by: rahulreddy15 <[email protected]> Co-authored-by: tharun0064 <[email protected]>
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.
No description provided.