Thanks for putting this together, Sourabh!
I made two comments in the document (error handling, with statement).

Are there any plans to support permissions (mode bits or acls) in the FS API?
I believe most (if not all) of the underlying filesystems support (some sort 
of) permission enforcement.

Thanks,
Tibor

> On Mar 17, 2017, at 10:14 PM, Sourabh Bajaj <[email protected]> 
> wrote:
> 
> Wanted to share the design proposal
> <https://docs.google.com/document/d/10qD0RXmdI0240wPShaGDRm9Zt9a_ess-ABlvYx2LZFA/edit#heading=h.czvx1winvche>
> for the Beam File System API in python. I have marked the places where it
> might be slightly different from the current Java implementation, mainly
> around error handling. As always feedback and comments are welcome.
> 
> Thanks
> Sourabh
> 
> On Wed, Mar 1, 2017 at 4:44 PM Chamikara Jayalath <[email protected]>
> wrote:
> 
>> Great! Thanks Sourabh.
>> 
>> - Cham
>> 
>> On Wed, Mar 1, 2017 at 3:58 PM Robert Bradshaw <[email protected]
>>> 
>> wrote:
>> 
>>> Much needed! Added a couple of comments.
>>> 
>>> On Wed, Mar 1, 2017 at 3:08 PM, Sourabh Bajaj <
>>> [email protected]> wrote:
>>> 
>>>> Hi,
>>>> 
>>>> BEAM-1441 <https://issues.apache.org/jira/browse/BEAM-1441> is a
>> ticket
>>>> for
>>>> implementing the Beam File System in the Python SDK similar to the one
>>>> introduced in BEAM-59 <https://issues.apache.org/jira/browse/BEAM-59>.
>> I
>>>> tried to take a pass on the implementation in #2136
>>>> <https://github.com/apache/beam/pull/2136> and followed the Java API
>> as
>>>> closely as possible. Please feel free to give your comments here or on
>>> the
>>>> pull request directly.
>>>> 
>>>> Reference: Original design doc
>>>> <https://docs.google.com/document/d/11TdPyZ9_zmjokhNWM3Id-
>>>> XJsVG3qel2lhdKTknmZ_7M/edit#>
>>>> 
>>>> 
>>>> Thanks
>>>> Sourabh
>>>> 
>>> 
>> 

Reply via email to