You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "Tsz-wo Sze (Jira)" <ji...@apache.org> on 2020/11/26 03:20:00 UTC
[jira] [Commented] (HDDS-4517) Ozone should only use public Ratis
APIs
[ https://issues.apache.org/jira/browse/HDDS-4517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17239044#comment-17239044 ]
Tsz-wo Sze commented on HDDS-4517:
----------------------------------
Usages of RaftServerProxy and RaftServerImpl:
{code}
-- ./hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/transport/server/ratis/ContainerStateMachine.java
74: import org.apache.ratis.server.impl.RaftServerProxy;
1 occurrence(s)
-- ./hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/transport/server/ratis/XceiverServerRatis.java
103: import org.apache.ratis.server.impl.RaftServerProxy;
1 occurrence(s)
-- ./hadoop-hdds/container-service/src/test/java/org/apache/hadoop/ozone/container/ContainerTestHelper.java
54: import org.apache.ratis.server.impl.RaftServerImpl;
55: import org.apache.ratis.server.impl.RaftServerProxy;
2 occurrence(s)
-- ./hadoop-ozone/insight/src/main/java/org/apache/hadoop/ozone/insight/datanode/RatisInsight.java
58: "org.apache.ratis.server.impl",
1 occurrence(s)
-- ./hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/container/TestHelper.java
50: import org.apache.ratis.server.impl.RaftServerImpl;
51: import org.apache.ratis.server.impl.RaftServerProxy;
2 occurrence(s)
-- ./hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/freon/TestHadoopDirTreeGenerator.java
29: import org.apache.ratis.server.impl.RaftServerImpl;
1 occurrence(s)
-- ./hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/freon/TestHadoopNestedDirGenerator.java
29: import org.apache.ratis.server.impl.RaftServerImpl;
1 occurrence(s)
-- ./hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/freon/TestOzoneClientKeyGenerator.java
28: import org.apache.ratis.server.impl.RaftServerImpl;
1 occurrence(s)
{code}
> Ozone should only use public Ratis APIs
> ---------------------------------------
>
> Key: HDDS-4517
> URL: https://issues.apache.org/jira/browse/HDDS-4517
> Project: Hadoop Distributed Data Store
> Issue Type: Improvement
> Reporter: Tsz-wo Sze
> Assignee: Tsz-wo Sze
> Priority: Major
>
> Ozone currently may use some internal Ratis APIs such as RaftServerProxy or even RaftServerImpl. The JIRA is to change Ozone to use only public Ratis APIs.
> This also helps Ratis to define the public APIs for state machines; see RATIS-1181.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org