[GitHub] [ozone] umamaheswararao commented on a change in pull request #3078: HDDS-6308: EC: Handle Replication Factor to consider EC Config in Recon UI

[GitBox <gi...@apache.org>]

umamaheswararao commented on a change in pull request #3078:
URL: https://github.com/apache/ozone/pull/3078#discussion_r806089259



##########
File path: hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/types/DatanodePipeline.java
##########
@@ -17,22 +17,22 @@
  */
 package org.apache.hadoop.ozone.recon.api.types;
 
-import org.apache.hadoop.hdds.client.ReplicationConfig;
-
 import java.util.UUID;
 
 /**
  * Metadata object that contains pipeline information of a Datanode.
  */
 public class DatanodePipeline {
   private UUID pipelineID;
-  private ReplicationConfig repConfig;
+  private String replicationType;
+  private String replicationFactor;
   private String leaderNode;
 
-  public DatanodePipeline(UUID pipelineID, ReplicationConfig repConfig,
-      String leaderNode) {
+  public DatanodePipeline(UUID pipelineID, String replicationType,

Review comment:
       I am not really sure, if that works. Let me try. Looks like this is some web object representation. If the fields and getter enough, then it should be ok, otherwise may fail (ex: interface IPipeline ). @avijayanhwx do you know, if that is ok?
   
   




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

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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