yihua commented on code in PR #18469:
URL: https://github.com/apache/hudi/pull/18469#discussion_r3037183789
##########
hudi-cli/src/main/java/org/apache/hudi/cli/utils/InputStreamConsumer.java:
##########
@@ -38,9 +38,8 @@ public InputStreamConsumer(InputStream is) {
@Override
public void run() {
- try {
- InputStreamReader isr = new InputStreamReader(is,
StandardCharsets.UTF_8);
- BufferedReader br = new BufferedReader(isr);
+ try (InputStreamReader isr = new InputStreamReader(is,
StandardCharsets.UTF_8);
+ BufferedReader br = new BufferedReader(isr)) {)
br.lines().forEach(log::info);
Review Comment:
🤖 There's a stray `)` at the end of this line — `)) {)` — which is a syntax
error and will prevent compilation. It should be `)) {` (no trailing paren).
This is confirmed by the CI failure on this PR. Could you drop that extra `)`
and push a fix?
--
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]
