hudi-agent commented on code in PR #19155:
URL: https://github.com/apache/hudi/pull/19155#discussion_r3521728141
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/HoodieBinaryCopyHandle.java:
##########
@@ -109,8 +109,7 @@ public HoodieBinaryCopyHandle(
}
public void write() {
- log.info("Start to merge source files " + this.inputFiles + " into target
file: " + this.path
- + ". Please pay attention that we will not rolling files based on
max-file-size config during binary copy.");
+ log.info("Start to merge source files {} into target file: {}. Please pay
attention that we will not rolling files based on max-file-size config during
binary copy.", this.inputFiles, this.path);
Review Comment:
🤖 nit: the original was split across two lines for readability; collapsing
it into a single ~200-char line makes it harder to scan. Could you keep the
format string and arguments on separate lines, e.g. `log.info("Start to merge
source files {} into target file: {}. ...",\n this.inputFiles, this.path)`?
<sub><i>⚠️ AI-generated; verify before applying. React 👍/👎 to flag
quality.</i></sub>
--
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]
