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

Release the incoming message when the decoding failed #409

Merged
merged 1 commit into from
Aug 14, 2018
Merged

Conversation

violetagg
Copy link
Member

Related to #406

@violetagg violetagg changed the base branch from master to 0.7.x August 6, 2018 11:38
@violetagg violetagg added this to the 0.7.9.RELEASE milestone Aug 6, 2018
@violetagg violetagg requested a review from smaldini August 6, 2018 11:39
@codecov-io
Copy link

Codecov Report

Merging #409 into 0.7.x will decrease coverage by 0.3%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##              0.7.x     #409      +/-   ##
============================================
- Coverage     69.05%   68.74%   -0.31%     
+ Complexity     1049     1043       -6     
============================================
  Files            72       72              
  Lines          4418     4419       +1     
  Branches        639      639              
============================================
- Hits           3051     3038      -13     
- Misses          979      988       +9     
- Partials        388      393       +5
Impacted Files Coverage Δ Complexity Δ
...actor/ipc/netty/http/server/HttpServerHandler.java 78.08% <100%> (-1.23%) 41 <0> (-1)
...tor/ipc/netty/channel/CloseableContextHandler.java 53.84% <0%> (-5.13%) 8% <0%> (-1%)
.../ipc/netty/channel/PooledClientContextHandler.java 63.86% <0%> (-4.21%) 26% <0%> (-1%)
...ctor/ipc/netty/resources/DefaultPoolResources.java 75% <0%> (-3.58%) 8% <0%> (-1%)
.../ipc/netty/http/server/HttpServerWSOperations.java 61.4% <0%> (-1.76%) 13% <0%> (-1%)
...in/java/reactor/ipc/netty/channel/FluxReceive.java 82.65% <0%> (-0.52%) 65% <0%> (-1%)

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 88462b0...c7c9132. Read the comment docs.

@smaldini smaldini merged commit 4b2f99a into 0.7.x Aug 14, 2018
@smaldini smaldini deleted the issue-406-1 branch August 14, 2018 22:09
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