You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/04 12:57:47 UTC
[GitHub] [beam] damccorm opened a new issue, #19741: Python local filesystem match does not work without directory separator.
damccorm opened a new issue, #19741:
URL: https://github.com/apache/beam/issues/19741
E.g. `beam.io.ReadFromText('./*.txt')` works but `beam.io.ReadFromText('*.txt')` throws a "No files found based on the filepattern..." error.
Imported from Jira [BEAM-7560](https://issues.apache.org/jira/browse/BEAM-7560). Original Jira may contain additional context.
Reported by: robertwb.
--
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: github-unsubscribe@beam.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] BjornPrime commented on issue #19741: Python local filesystem match does not work without directory separator.
Posted by GitBox <gi...@apache.org>.
BjornPrime commented on issue #19741:
URL: https://github.com/apache/beam/issues/19741#issuecomment-1176398172
.take-issue
--
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: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] BjornPrime commented on issue #19741: Python local filesystem match does not work without directory separator.
Posted by GitBox <gi...@apache.org>.
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: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org