sravani-revuri opened a new pull request, #9741:
URL: https://github.com/apache/ozone/pull/9741

   
   
   ## What changes were proposed in this pull request?
   ContainerInfo#getProtobuf there are duplicates setter invocations for the 
following.
   
   .setContainerID(getContainerID())-: Line 291 and Line 295 is called twice. 
https://github.com/apache/ozone/blob/08260d2a24a30ba62573a70415b391791bb64eb7/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerInfo.java#L267-L274
   .setReplicationType(getReplicationType()) -: Line 299 and Line 310
   
https://github.com/apache/ozone/blob/08260d2a24a30ba62573a70415b391791bb64eb7/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerInfo.java#L278-L288
   Removed the duplicate ones.
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-14573
   
   ## How was this patch tested?
   
   Tested with container info command
   ```vrevuri@KJ3K6CH6Q1 ozone % docker compose exec scm ozone admin container 
info 2
   Container id: 2
   Pipeline id: 113c742c-18f2-4260-8fcb-fc970fa3ab03
   Write PipelineId: 113c742c-18f2-4260-8fcb-fc970fa3ab03
   Write Pipeline State: OPEN
   Container State: OPEN
   SequenceId: 0
   Datanodes: 
[83d513a6-3513-4bd1-9077-f120d190049b/ozone-datanode-1.ozone_default]
   Replicas: [State: OPEN; ReplicaIndex: 0; SequenceId: 0; Origin: 
83d513a6-3513-4bd1-9077-f120d190049b; Location: 
83d513a6-3513-4bd1-9077-f120d190049b/ozone-datanode-1.ozone_default]```
   


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

Reply via email to