szetszwo commented on pull request #596: URL: https://github.com/apache/ratis/pull/596#issuecomment-1034885140
@guohao-rosicky , It seems the server behavior was changed. The primary did not forward the request to the next server as shown below that s1 returns null; see https://issues.apache.org/jira/secure/attachment/13039878/596_debug.patch ``` 2022-02-10 20:37:26,654 [Time-limited test] INFO datastream.DataStreamTestUtils (DataStreamTestUtils.java:writeAndCloseAndAssertReplies(314)) - start Stream0 2022-02-10 20:37:26,683 [nioEventLoopGroup-3-1] INFO statemachine.StateMachine (DataStreamTestUtils.java:stream(151)) - XXX MultiDataStreamStateMachine:s0:group-56A12F14DED8 put 0@client-E43970695D43, SingleDataStream: writeRequest=RaftClientRequest:client-E43970695D43->s0@group-56A12F14DED8, cid=0, seq=0, DataStream, Message:<EMPTY>, logEntry=null 2022-02-10 20:37:26,736 [Time-limited test] INFO datastream.DataStreamTestUtils (DataStreamTestUtils.java:assertHeader(331)) - XXX s0: dataSize=7478599, stepDownLeader=false, header=RaftClientRequest:client-E43970695D43->s0@group-56A12F14DED8, cid=0, seq=0, DataStream, null 2022-02-10 20:37:26,736 [Time-limited test] INFO statemachine.StateMachine (DataStreamTestUtils.java:getSingleDataStream(181)) - XXX MultiDataStreamStateMachine:s0:group-56A12F14DED8: get 0@client-E43970695D43 return SingleDataStream: writeRequest=RaftClientRequest:client-E43970695D43->s0@group-56A12F14DED8, cid=0, seq=0, DataStream, Message:<EMPTY>, logEntry=null 2022-02-10 20:37:26,737 [Time-limited test] INFO datastream.DataStreamTestUtils (DataStreamTestUtils.java:assertHeader(331)) - XXX s1: dataSize=7478599, stepDownLeader=false, header=RaftClientRequest:client-E43970695D43->s0@group-56A12F14DED8, cid=0, seq=0, DataStream, null 2022-02-10 20:37:26,737 [Time-limited test] INFO statemachine.StateMachine (DataStreamTestUtils.java:getSingleDataStream(181)) - XXX MultiDataStreamStateMachine:s1:group-56A12F14DED8: get 0@client-E43970695D43 return null ``` Please check if the new behavior is correct. If yes, we can update the test. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@ratis.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
