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

Fix disruptor : fix the problem of hold event data in consume event #1858

Merged
merged 1 commit into from
Aug 4, 2021

Conversation

Switch-vov
Copy link
Contributor

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Sorry, something went wrong.

@Switch-vov
Copy link
Contributor Author

@yu199195 I will commit some Benchmarks later. Maybe next commit.

@codecov-commenter
Copy link

codecov-commenter commented Aug 3, 2021

Codecov Report

Merging #1858 (5b6662a) into master (b9458df) will decrease coverage by 0.10%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1858      +/-   ##
============================================
- Coverage     66.13%   66.03%   -0.11%     
  Complexity     2585     2585              
============================================
  Files           489      489              
  Lines         10972    10972              
  Branches       1076     1076              
============================================
- Hits           7256     7245      -11     
- Misses         3160     3172      +12     
+ Partials        556      555       -1     
Impacted Files Coverage Δ
...che/shenyu/sync/data/http/HttpSyncDataService.java 65.04% <0.00%> (-9.71%) ⬇️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 63.15% <0.00%> (-1.06%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9458df...5b6662a. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants