deepakpanda93 commented on code in PR #19494:
URL: https://github.com/apache/hudi/pull/19494#discussion_r3712533141
##########
hudi-common/src/main/java/org/apache/hudi/common/util/RetryHelper.java:
##########
@@ -122,6 +127,20 @@ public T start() throws R {
return start(this.func);
}
+ /**
+ * Renders an exception as a single line, keeping the root cause so that the
warning stays
+ * actionable without the whole stack trace.
+ */
+ @VisibleForTesting
+ static String summarise(Throwable t) {
+ Throwable rootCause = t;
Review Comment:
Fair point, renamed to `summarize` in both the helper and its test.
Checked before changing it rather than assuming: the codebase had one file
using the American spelling against the two I had just added with the British
one, and the JDK itself is `Collectors.summarizingInt` /
`DoubleSummaryStatistics`. No occurrences of `summarise` remain.
--
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]
