This is an automated email from the ASF dual-hosted git repository.
albumenj pushed a commit to branch 3.3
in repository https://gitbox.apache.org/repos/asf/dubbo.git
The following commit(s) were added to refs/heads/3.3 by this push:
new 4febe492ea fix: Remove invalid parameters (#14507)
4febe492ea is described below
commit 4febe492eaa57e58b92bb27b42144ac7bea772a9
Author: Zaki <[email protected]>
AuthorDate: Mon Aug 12 09:53:36 2024 +0800
fix: Remove invalid parameters (#14507)
Co-authored-by: Albumen Kevin <[email protected]>
---
.../java/org/apache/dubbo/rpc/protocol/tri/call/TripleClientCall.java | 2 +-
.../dubbo/rpc/protocol/tri/stream/AbstractTripleClientStream.java | 2 +-
.../java/org/apache/dubbo/rpc/protocol/tri/stream/ClientStream.java | 3 +--
.../apache/dubbo/rpc/protocol/tri/stream/TripleClientStreamTest.java | 2 +-
4 files changed, 4 insertions(+), 5 deletions(-)
diff --git
a/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/call/TripleClientCall.java
b/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/call/TripleClientCall.java
index 877fcd09d7..24b80585a8 100644
---
a/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/call/TripleClientCall.java
+++
b/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/call/TripleClientCall.java
@@ -191,7 +191,7 @@ public class TripleClientCall implements ClientCall,
ClientStream.Listener {
data = requestMetadata.packableMethod.packRequest(message);
int compressed =
Identity.MESSAGE_ENCODING.equals(requestMetadata.compressor.getMessageEncoding())
? 0 : 1;
final byte[] compress = requestMetadata.compressor.compress(data);
- stream.sendMessage(compress, compressed, false).addListener(f -> {
+ stream.sendMessage(compress, compressed).addListener(f -> {
if (!f.isSuccess()) {
cancelByLocal(f.cause());
}
diff --git
a/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/AbstractTripleClientStream.java
b/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/AbstractTripleClientStream.java
index e1fdd0f14a..7ceea89e60 100644
---
a/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/AbstractTripleClientStream.java
+++
b/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/AbstractTripleClientStream.java
@@ -146,7 +146,7 @@ public abstract class AbstractTripleClientStream extends
AbstractStream implemen
}
@Override
- public ChannelFuture sendMessage(byte[] message, int compressFlag, boolean
eos) {
+ public ChannelFuture sendMessage(byte[] message, int compressFlag) {
ChannelFuture checkResult = preCheck();
if (!checkResult.isSuccess()) {
return checkResult;
diff --git
a/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/ClientStream.java
b/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/ClientStream.java
index 304d3c4653..4f9f8c8b9b 100644
---
a/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/ClientStream.java
+++
b/dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/stream/ClientStream.java
@@ -62,10 +62,9 @@ public interface ClientStream extends Stream {
* Send message to remote peer.
*
* @param message message to send to remote peer
- * @param eos whether this is the last message
* @return future to callback when send message is done
*/
- Future<?> sendMessage(byte[] message, int compressFlag, boolean eos);
+ Future<?> sendMessage(byte[] message, int compressFlag);
/**
* No more data will be sent, half close this stream to wait server
response.
diff --git
a/dubbo-rpc/dubbo-rpc-triple/src/test/java/org/apache/dubbo/rpc/protocol/tri/stream/TripleClientStreamTest.java
b/dubbo-rpc/dubbo-rpc-triple/src/test/java/org/apache/dubbo/rpc/protocol/tri/stream/TripleClientStreamTest.java
index d3d16c9946..1f8522305d 100644
---
a/dubbo-rpc/dubbo-rpc-triple/src/test/java/org/apache/dubbo/rpc/protocol/tri/stream/TripleClientStreamTest.java
+++
b/dubbo-rpc/dubbo-rpc-triple/src/test/java/org/apache/dubbo/rpc/protocol/tri/stream/TripleClientStreamTest.java
@@ -100,7 +100,7 @@ class TripleClientStreamTest {
verify(writeQueue).enqueueFuture(any(HeaderQueueCommand.class),
any(Executor.class));
// no other commands
verify(writeQueue).enqueue(any(QueuedCommand.class));
- stream.sendMessage(new byte[0], 0, false);
+ stream.sendMessage(new byte[0], 0);
verify(writeQueue).enqueueFuture(any(DataQueueCommand.class),
any(Executor.class));
verify(writeQueue, times(2)).enqueueFuture(any(QueuedCommand.class),
any(Executor.class));
stream.halfClose();