Cribbee commented on PR #29219: URL: https://github.com/apache/flink/pull/29219#issuecomment-5724950350
Thanks for reviewing. I revisited FLINK-6417 and #15436, and I agree that moving the logic into FileSource does not by itself address the compatibility concerns. The current implementation still reinterprets literal paths containing glob characters and does not fully preserve subclass/filter behavior. I propose keeping the existing enumerators unchanged and making glob expansion explicitly opt-in via `setFileEnumerator(...)`. The implementation would also need to avoid traversing unrelated subtrees and handle filtering and traversal errors correctly. Would you be open to this direction for FileSource? I’ll also address the architecture-test failure -- 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]
