sodonnel commented on a change in pull request #3078:
URL: https://github.com/apache/ozone/pull/3078#discussion_r806094808
##########
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:
If we cannot store the ReplicationConfig object in the class, perhaps we
could pass ReplicationConfig to the constructor, and inside the constructor
pull out the String values and store them? In general, we are trying to get
away from Factor and Type in the API everywhere, so that would be a good
compromise perhaps?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
