ayushtkn commented on code in PR #4286:
URL: https://github.com/apache/ozone/pull/4286#discussion_r1122854240


##########
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/freon/RandomKeyGenerator.java:
##########
@@ -484,8 +484,9 @@ private void printStats(PrintStream out) {
     out.println("Number of Volumes created: " + numberOfVolumesCreated);
     out.println("Number of Buckets created: " + numberOfBucketsCreated);
     out.println("Number of Keys added: " + numberOfKeysAdded);
-    out.println("Replication: " + replicationConfig.getReplication());
-    out.println("Replication type: " + replicationConfig.getReplicationType());
+    if (replicationConfig != null) {
+      out.println("Replication: " + replicationConfig);
+    }

Review Comment:
   But still why to remove a line from the CLI which ain't creating any trouble 
and that too in scope of fixing a NPE.
   anyway the change is in by the time I could circle back, should be fine 
then.....



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