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.

Sign up for GitHub

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

[ISSUE #4701]Fix use tcp protocol client send message, it throw a DecoderException #4702

Merged
merged 7 commits into from
Jan 10, 2024

optimize code and add some comments

32656bc
Select commit
Loading
Failed to load commit list.
Merged

[ISSUE #4701]Fix use tcp protocol client send message, it throw a DecoderException #4702

optimize code and add some comments
32656bc
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 8, 2024 in 0s

62.50% of diff hit (target 17.39%)

View this Pull Request on Codecov

62.50% of diff hit (target 17.39%)

Annotations

Check warning on line 46 in eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java#L46 

Added line #L46 was not covered by tests

Check warning on line 138 in eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java#L138 

Added line #L138 was not covered by tests

Check warning on line 155 in eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java#L153-L155 

Added lines #L153 - L155 were not covered by tests

Check warning on line 162 in eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eventmesh-common/src/main/java/org/apache/eventmesh/common/protocol/tcp/codec/Codec.java#L162 

Added line #L162 was not covered by tests

Check warning on line 113 in eventmesh-sdks/eventmesh-sdk-java/src/main/java/org/apache/eventmesh/client/tcp/common/TcpClient.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eventmesh-sdks/eventmesh-sdk-java/src/main/java/org/apache/eventmesh/client/tcp/common/TcpClient.java#L113 

Added line #L113 was not covered by tests
View more details on Codecov