bd2019us commented on a change in pull request #4116: fix issue #4115, change
replaceAll() to replace
URL: https://github.com/apache/incubator-pinot/pull/4116#discussion_r278292114
##########
File path:
thirdeye/thirdeye-hadoop/src/main/java/org/apache/pinot/thirdeye/hadoop/push/DeleteOverlappingSegmentsInPinot.java
##########
@@ -79,7 +79,7 @@ private static IdealState
computeNewIdealStateAfterDeletingOverlappingSegments(H
}
}
LOG.info("HOURLY segments that can be deleted: {}",
hourlySegmentsToDelete.size());
- LOG.info("Hourly segments to delete {}",
hourlySegmentsToDelete.toString().replaceAll(",", "\n"));
+ LOG.info("Hourly segments to delete {}",
hourlySegmentsToDelete.toString().replace(",", "\n"));
Review comment:
Sorry for my late reply. I optimized the string to char instead. Please
check. 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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
