amaliujia edited a comment on pull request #298:
URL: https://github.com/apache/incubator-ratis/pull/298#issuecomment-736130756


   I have identified the reason for failing to apply the transaction: 
https://github.com/apache/incubator-ratis/blob/ebabdbf570b50af0d868a5f4e5acf6fe5adc4876/ratis-examples/src/main/java/org/apache/ratis/examples/filestore/FileStoreStateMachine.java#L94
   
   The request in STREAM header will be sent to leader, and hits this line. 
Because now we have allowed a customized message in that request, so need to 
update here as well to parse the proper proto. 
   
   ----------------
   
   actually no... the `StreamWriteRequestHeaderProto`  is added to the example 
proto so this function is supposed to be changed. 
   
   I am guessing that this request is not the request in stream header but 
still checking. 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to