kerneltime commented on code in PR #8050:
URL: https://github.com/apache/ozone/pull/8050#discussion_r1989816086
##########
hadoop-hdds/tools/src/main/java/org/apache/hadoop/hdds/scm/cli/container/ListSubcommand.java:
##########
@@ -90,11 +95,35 @@ public class ListSubcommand extends ScmSubcommand {
}
- private void outputContainerInfo(ContainerInfo containerInfo)
- throws IOException {
- // Print container report info.
+ private void outputContainerInfo(ContainerInfo containerInfo) throws
IOException {
+ // Original behavior - just print the container JSON
System.out.println(WRITER.writeValueAsString(containerInfo));
}
+
+ private void outputContainerInfoAsJsonMember(ContainerInfo containerInfo,
boolean isFirst,
+ boolean isLast) throws IOException {
+ // JSON array format with proper brackets and commas
+ if (isFirst) {
+ // Start of array
+ System.out.print("[");
Review Comment:
Will look into SequenceWriter
--
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-unsubscr...@ozone.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org
For additional commands, e-mail: issues-h...@ozone.apache.org
