ArnavBalyan commented on code in PR #11608:
URL: https://github.com/apache/pinot/pull/11608#discussion_r1330145418
##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java:
##########
@@ -782,7 +782,7 @@ public SuccessResponse reloadAllSegments(
LOGGER.error("Failed to add reload all segments job meta into
zookeeper for table: {}", tableNameWithType, e);
}
}
- return new SuccessResponse("Segment reload details: " +
JsonUtils.objectToString(perTableMsgData));
+ return new
SuccessResponse(JsonUtils.objectToPrettyString(perTableMsgData));
Review Comment:
For now, I've just kept the json string in the success response which makes
it easier to parse/understand. Please let me know what you think Thanks!
--
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]
