hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tsz Wo Nicholas Sze (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-12720) Ozone: Ratis options are not passed from KSM Client protobuf helper correctly.
Date Mon, 30 Oct 2017 18:35:00 GMT

    [ https://issues.apache.org/jira/browse/HDFS-12720?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16225504#comment-16225504

Tsz Wo Nicholas Sze commented on HDFS-12720:

Patch looks good.  Just some minor comments:

- Should the code below be channel().close().sync()?
      try {
      } catch (InterruptedException ie) {
        LOG.error("close for pipeline:{} interrupted", pipeline, ie);
-* Also, the ie won't be printed to LOG.  It should be chanaged to:
        LOG.warn("close for pipeline: " + pipeline + " interrupted", ie);
-* Since the ie is ignored, let's use warn instead of error.

- Let's add a newRaftGroup(datanodes) to RatisHelper
  static <E extends DatanodeID> RaftGroup newRaftGroup(List<E> datanodes) {
    return newRaftGroup(toRaftPeers(datanodes));
so that we can use it in XceiverClientRatis
  public void createPipeline(String clusterId, List<DatanodeID> datanodes)
      throws IOException {
    final RaftGroup group = RatisHelper.newRaftGroup(datanodes);
    LOG.debug("initializing pipeline:{} with nodes:{}", clusterId, group.getPeers());
    reinitialize(datanodes, group);

> Ozone: Ratis options are not passed from KSM Client protobuf helper correctly.
> ------------------------------------------------------------------------------
>                 Key: HDFS-12720
>                 URL: https://issues.apache.org/jira/browse/HDFS-12720
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>    Affects Versions: HDFS-7240
>            Reporter: Mukul Kumar Singh
>            Assignee: Mukul Kumar Singh
>              Labels: ozoneMerge
>             Fix For: HDFS-7240
>         Attachments: HDFS-12720-HDFS-7240.001.patch, HDFS-12720-HDFS-7240.002.patch,
> {{KeySpaceManagerProtocolClientSideTranslatorPB#allocateBlock}} and {{KeySpaceManagerProtocolClientSideTranslatorPB#openKey}}
do not pass the ratis replication factor and replication type to the KSM server. this causes
the allocations using ratis model to resort to standalone mode even when Ratis mode is specified.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org

View raw message