DanielLeens commented on PR #10950: URL: https://github.com/apache/seatunnel/pull/10950#issuecomment-4642242440
Thanks, @dybyte — I agree with that concern. I rechecked the PR state, and the head is still unchanged at `9de9ae5`, so Daniel's previous review conclusion stays the same from my side: improving diagnosability is useful, but the current patch still needs to avoid unconditional INFO-level per-split logging on the normal split-generation path before it is ready to merge. There is no new code to rereview yet, so I am not opening a fresh full review in this round. Happy to take another full pass once the author pushes an update. -- 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]
