This is an automated email from the ASF dual-hosted git repository.
raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new e109ce16ea GH-46121: [Python] Add missing `column_index` argument to
`ArrowReaderProperties::read_dictionary`'s Cython binding (#46122)
e109ce16ea is described below
commit e109ce16eaa69dc212fe3f474047da7d65902245
Author: Lester Fan <[email protected]>
AuthorDate: Mon Apr 14 05:46:25 2025 -0400
GH-46121: [Python] Add missing `column_index` argument to
`ArrowReaderProperties::read_dictionary`'s Cython binding (#46122)
### Rationale for this change
This PR adds the missing `column_index` argument to
`ArrowReaderProperties::read_dictionary`'s Cython binding to resolve the issue
described in https://github.com/apache/arrow/issues/46121.
### What changes are included in this PR?
1. Add the missing `column_index` argument to
`ArrowReaderProperties::read_dictionary`'s Cython binding.
### Are these changes tested?
No (I'm not sure where the tests for Cython bindings are; I'm happy to add
a test if someone can point me in the right direction)
### Are there any user-facing changes?
Yes; this is a bug fix to a user facing API (though it's admittedly
unlikely to be used).
* GitHub Issue: #46121
Authored-by: Lester Fan <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
---
python/pyarrow/_parquet.pxd | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/python/pyarrow/_parquet.pxd b/python/pyarrow/_parquet.pxd
index e6de9712f8..f7df8a052a 100644
--- a/python/pyarrow/_parquet.pxd
+++ b/python/pyarrow/_parquet.pxd
@@ -395,7 +395,7 @@ cdef extern from "parquet/api/reader.h" namespace "parquet"
nogil:
cdef cppclass ArrowReaderProperties:
ArrowReaderProperties()
void set_read_dictionary(int column_index, c_bool read_dict)
- c_bool read_dictionary()
+ c_bool read_dictionary(int column_index)
void set_batch_size(int64_t batch_size)
int64_t batch_size()
void set_pre_buffer(c_bool pre_buffer)