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

[#9224] Add PinotAsyncTemplate #9225

Merged
merged 2 commits into from
Sep 26, 2022

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Sep 23, 2022

No description provided.

@emeroad emeroad added this to the 2.5.0 milestone Sep 23, 2022
@emeroad emeroad linked an issue Sep 23, 2022 that may be closed by this pull request
@emeroad emeroad force-pushed the #9224_PinotAsyncTemplate branch from 80cd6bf to 79bf62e Compare September 23, 2022 10:44
@codecov
Copy link

codecov bot commented Sep 23, 2022

Codecov Report

Merging #9225 (361af23) into master (a318f2e) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #9225      +/-   ##
============================================
- Coverage     39.85%   39.79%   -0.06%     
+ Complexity    11774    11772       -2     
============================================
  Files          3364     3367       +3     
  Lines         90762    90905     +143     
  Branches      10093    10096       +3     
============================================
+ Hits          36174    36178       +4     
- Misses        51499    51638     +139     
  Partials       3089     3089              
Impacted Files Coverage Δ
...inpoint/metric/common/pinot/ParameterRecorder.java 0.00% <0.00%> (ø)
...npoint/metric/common/pinot/PinotAsyncTemplate.java 0.00% <0.00%> (ø)
.../pinpoint/metric/common/pinot/PinotDataSource.java 45.45% <ø> (-0.38%) ⬇️
...pinpoint/metric/common/pinot/StatementWrapper.java 0.00% <0.00%> (ø)
.../pinpoint/metric/common/pinot/TransformFuture.java 0.00% <0.00%> (ø)
...nt/metric/common/pinot/WrappedPinotConnection.java 9.09% <0.00%> (-0.33%) ⬇️
...tric/web/dao/pinot/PinotSystemMetricDoubleDao.java 0.00% <0.00%> (ø)
...metric/web/dao/pinot/PinotSystemMetricLongDao.java 0.00% <0.00%> (ø)
...etric/web/service/SystemMetricDataServiceImpl.java 0.00% <0.00%> (ø)
...pinpoint/rpc/ChannelWriteFailListenableFuture.java 42.85% <0.00%> (-28.58%) ⬇️
... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@emeroad emeroad force-pushed the #9224_PinotAsyncTemplate branch 3 times, most recently from b2d3e93 to 361af23 Compare September 26, 2022 08:29
@emeroad emeroad force-pushed the #9224_PinotAsyncTemplate branch from 361af23 to ba2dcf3 Compare September 26, 2022 08:49
@emeroad emeroad merged commit 15c2a11 into pinpoint-apm:master Sep 26, 2022
@emeroad emeroad deleted the #9224_PinotAsyncTemplate branch September 28, 2022 02:53
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.

Add non-blocking async Mybatis template for Pinot
1 participant