Jackie-Jiang commented on code in PR #11608:
URL: https://github.com/apache/pinot/pull/11608#discussion_r1328167943
##########
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:
It doesn't need to be a json, maybe simply
```suggestion
return new SuccessResponse("Segment reload details: " + perTableMsgData);
```
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
