BjornPrime commented on issue #19741: URL: https://github.com/apache/beam/issues/19741#issuecomment-1178065277
In the comments of FileSystems.match (line 190 in filesystems.py), it explicitly says that they're mostly using fnmatch syntax but for some reason altered it such that '*' excludes '/', which I'm thinking is why you currently need to include '/' explicitly. Is there some reason we aren't just using fnmatch, or at least conforming to their syntax? The comment doesn't give any rationale for that design decision. -- 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]
