You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ratis.apache.org by GitBox <gi...@apache.org> on 2020/11/30 22:03:51 UTC

[GitHub] [incubator-ratis] amaliujia edited a comment on pull request #298: RATIS-1111. Change the FileStore example to use Streaming

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


   @szetszwo sorry for a bit delay on this PR.
   
   1. Until this moment I am still not sure what `DataApi.link(..)` is supposed to do for `FileStoreStateMachine`. Any hint for it? Given a DataStream and a LogEntryProto, what's the required operation to link these two objects?
   
   2. I updated the unit test and found that transaction cannot be applied successfully. The debugger did not stop at `DataApi.link(..)` so it might not because that I haven't implemented the `DataApi.link` API. I am still debugging why it cannot apply transaction successfully (async API is a bit hard to debug) and will update when I find something interesting. 


----------------------------------------------------------------
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:
users@infra.apache.org