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

[#11166] Bypass cache for long SQL queries #11177

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

kojandy
Copy link
Contributor

@kojandy kojandy commented Jun 21, 2024

Resolves #11166.

@kojandy kojandy self-assigned this Jun 21, 2024
@kojandy kojandy force-pushed the feature/11166-sql-max-length branch from 7ed8d1e to 2bbdbac Compare June 26, 2024 09:56
Copy link

codecov bot commented Jun 26, 2024

Codecov Report

Attention: Patch coverage is 75.60976% with 10 lines in your changes missing coverage. Please review.

Project coverage is 37.00%. Comparing base (c42084f) to head (255e510).
Report is 9 commits behind head on master.

Files Patch % Lines
...int/profiler/metadata/UidCachingSqlNormalizer.java 79.31% 3 Missing and 3 partials ⚠️
...npoint/bootstrap/config/DefaultProfilerConfig.java 33.33% 2 Missing ⚠️
.../provider/metadata/SqlMetadataServiceProvider.java 33.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11177      +/-   ##
============================================
- Coverage     37.03%   37.00%   -0.03%     
+ Complexity    12052    12046       -6     
============================================
  Files          3974     3978       +4     
  Lines         97808    97859      +51     
  Branches      10410    10414       +4     
============================================
- Hits          36219    36212       -7     
- Misses        58615    58673      +58     
  Partials       2974     2974              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kojandy kojandy force-pushed the feature/11166-sql-max-length branch from 2bbdbac to aae7860 Compare July 1, 2024 06:20
@kojandy kojandy force-pushed the feature/11166-sql-max-length branch from aae7860 to 255e510 Compare July 1, 2024 08:37
@kojandy kojandy marked this pull request as ready for review July 1, 2024 08:38
@kojandy kojandy merged commit 221ced9 into pinpoint-apm:master Jul 1, 2024
2 of 3 checks passed
@kojandy kojandy deleted the feature/11166-sql-max-length branch July 1, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bypass cache for long SQL queries
1 participant