[ 
https://issues.apache.org/jira/browse/BEAM-13493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17517592#comment-17517592
 ] 

Beam JIRA Bot commented on BEAM-13493:
--------------------------------------

This issue was marked "stale-P2" and has not received a public comment in 14 
days. It is now automatically moved to P3. If you are still affected by it, you 
can comment and move it back to P2.

> fileio calls mkdirs with basepath and not dirname(full_file_name)
> -----------------------------------------------------------------
>
>                 Key: BEAM-13493
>                 URL: https://issues.apache.org/jira/browse/BEAM-13493
>             Project: Beam
>          Issue Type: Bug
>          Components: io-py-files
>    Affects Versions: 2.34.0, 2.35.0
>            Reporter: Javier Domingo Cansino
>            Priority: P3
>
> When calling {{apache_beam.io.fileio.WriteToFiles}} with a {{file_naming}} 
> argument that adds a directory to the path, the current implementation fails 
> to write files if a {{mkdirs}} or analogous call is needed in the underlying 
> file storage.
> Example, 
> {code}
> apache_beam.io.fileio.WriteToFiles(
>     path="some/base/dir", sink=..., destination=lambda x: "events",
>     file_naming=lambda *x: "subdir/file.txt"
> ) 
> {code}
> the current fileio implementation will call {{mkdirs}} with {{some/base/dir}} 
> instead of {{some/base/dir/subdir}}.
> The bug is currently at 
> https://github.com/apache/beam/blob/67bcf1e16e3fdf68cdea7a4b42b9c003e4b8948c/sdks/python/apache_beam/io/fileio.py#L605.
> ====
> Personally, I would recommend changing the FileSystems interface to have 
> `open` call `mkdirs` in storages that require root parent directory creation.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to