[ 
https://issues.apache.org/jira/browse/ARROW-11464?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Pac A. He updated ARROW-11464:
------------------------------
    Description: 
The {{*pyarrow.parquet.read_pandas*}} 
[implementation|https://github.com/apache/arrow/blob/816c23af4478fe28f31d474e90b433baeadb7a78/python/pyarrow/parquet.py#L1740-L1754]
 doesn't conform to its 
[docs|https://arrow.apache.org/docs/python/generated/pyarrow.parquet.read_pandas.html#pyarrow.parquet.read_pandas]
 in at least these ways:
 # The docs state that a *{{filesystem}}* option can be provided, as it should 
be. Without this option I cannot read from S3, etc. The implementation, 
however, doesn't have this option! As such I currently cannot use it to read 
from S3!
 # The docs (_not in the type definition!_) state that the default value for 
*{{use_legacy_dataset}}* is False, whereas the implementation has a value of 
True. Actually, however, if a value of True is used, then I can't read a 
partitioned dataset at all, so in this case maybe its the doc that needs to 
change to True.

It looks to have been implemented and reviewed pretty carelessly.

 

  was:
The {{*pyarrow.parquet.read_pandas*}} 
[implementation|https://github.com/apache/arrow/blob/816c23af4478fe28f31d474e90b433baeadb7a78/python/pyarrow/parquet.py#L1740-L1754]
 doesn't conform to its 
[docs|https://arrow.apache.org/docs/python/generated/pyarrow.parquet.read_pandas.html#pyarrow.parquet.read_pandas]
 in at least these ways:
 # The docs state that a *{{filesystem}}* option can be provided, as it should 
be. Without this option I cannot read from S3, etc. The implementation, 
however, doesn't have this option! As such I currently cannot use it to read 
from S3!
 # The docs (not in the type definition!) state that the default value for 
*{{use_legacy_dataset}}* is False, whereas the implementation has a value of 
True. Actually, however, if a value of True is used, then I can't read a 
partitioned dataset at all, so in this case maybe its the doc that needs to 
change to True.

It looks to have been implemented and reviewed pretty carelessly.

 


> [Python] pyarrow.parquet.read_pandas doesn't conform to its docs
> ----------------------------------------------------------------
>
>                 Key: ARROW-11464
>                 URL: https://issues.apache.org/jira/browse/ARROW-11464
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Python
>    Affects Versions: 3.0.0
>         Environment: latest
>            Reporter: Pac A. He
>            Priority: Major
>
> The {{*pyarrow.parquet.read_pandas*}} 
> [implementation|https://github.com/apache/arrow/blob/816c23af4478fe28f31d474e90b433baeadb7a78/python/pyarrow/parquet.py#L1740-L1754]
>  doesn't conform to its 
> [docs|https://arrow.apache.org/docs/python/generated/pyarrow.parquet.read_pandas.html#pyarrow.parquet.read_pandas]
>  in at least these ways:
>  # The docs state that a *{{filesystem}}* option can be provided, as it 
> should be. Without this option I cannot read from S3, etc. The 
> implementation, however, doesn't have this option! As such I currently cannot 
> use it to read from S3!
>  # The docs (_not in the type definition!_) state that the default value for 
> *{{use_legacy_dataset}}* is False, whereas the implementation has a value of 
> True. Actually, however, if a value of True is used, then I can't read a 
> partitioned dataset at all, so in this case maybe its the doc that needs to 
> change to True.
> It looks to have been implemented and reviewed pretty carelessly.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to