Re: [PR] [FLINK-6417][connector/filesystem] Add opt-in glob enumeration for FileSource [flink]

["Cribbee (via GitHub)" <gi...@apache.org>]

Cribbee commented on PR #29219:
URL: https://github.com/apache/flink/pull/29219#issuecomment-5729774552

   Updated in a7ccd2010b6: the existing enumerators are unchanged, and glob expansion is explicitly opt-in through `setFileEnumerator(...)`. Traversal is prefix-pruned; the delegate receives unique concrete files for filtering and splitting. Directory-filter and subclass-hook boundaries are documented and tested.
   
   Local connector `verify`, HDFS/native S3 integration tests, and production architecture checks pass. The original internal-API dependency is gone; a separate commit fixes the architecture rule's public-array false positives while retaining internal-array checks. Full CI is still pending.
   
   This adds a `@PublicEvolving` API, so the FLIP prerequisite in `AGENTS.md` remains open. Could you advise on the FLIP scope and whether the architecture-rule fix should be tracked separately? Thank you.
   


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org