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

Posted to issues@ratis.apache.org by "Tsz-wo Sze (Jira)" <ji...@apache.org> on 2021/12/15 14:48:00 UTC

[jira] [Commented] (RATIS-1471) The Constructor of RaftStorageImpl should be backward compatible

    [ https://issues.apache.org/jira/browse/RATIS-1471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17460007#comment-17460007 ] 

Tsz-wo Sze commented on RATIS-1471:
-----------------------------------

bq. Here is the code ...

I see.  It uses RaftStorage to access the log files.

As mentioned, RaftStorage is not a public API.  Alluxio should not use it.  I am fine to change the Ratis code for this time but we may not be able to do the same in the future.

Could you also add a new class RaftStorage.Builder in the pull request so that Alluxio can use it?  You may see RaftServer.Builder as an example.

> The Constructor of RaftStorageImpl should be backward compatible
> ----------------------------------------------------------------
>
>                 Key: RATIS-1471
>                 URL: https://issues.apache.org/jira/browse/RATIS-1471
>             Project: Ratis
>          Issue Type: Bug
>            Reporter: Hui Fei
>            Assignee: Hui Fei
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Alluxio depends on ratis. When I want to upgrade ratis version, there are compatible issues.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)