DanielLeens commented on PR #10950: URL: https://github.com/apache/seatunnel/pull/10950#issuecomment-4648034017
Thanks all — I think @yzeng1618's hybrid direction is the right way to thread the needle here. On the current unchanged head `9de9ae5`, Daniel's source-level conclusion is still the same: the blocker is not "more visibility" in general, it is the current shape of unconditional per-split `INFO` logging on the normal split-generation path. A summary at `INFO` plus detailed per-split SQL / bound values at `DEBUG` (or behind an explicit troubleshooting switch) would address the operational need without turning large snapshot jobs into log floods. Since there is still no new commit after the last Daniel review on this head, I am keeping this as a reply on the current discussion rather than opening a fresh full review record. If the author pushes an updated patch along that hybrid line, I am happy to re-review the new head. -- 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]
