Copilot commented on code in PR #45998:
URL: https://github.com/apache/arrow/pull/45998#discussion_r4036438379
##########
python/pyarrow/includes/libarrow_fs.pxd:
##########
@@ -288,6 +287,12 @@ cdef extern from "arrow/filesystem/api.h" namespace
"arrow::fs" nogil:
void ConfigureKerberosTicketCachePath(c_string path)
void ConfigureExtraConf(c_string key, c_string value)
+ const c_string& host() const
+ int port() const
+ const c_string& user() const
+ const c_string& kerb_ticket() const
+ const unordered_map[c_string, c_string]&extra_conf() const
Review Comment:
The Cython declaration for `extra_conf()` is missing a space between the
reference declarator and the method name (`]&extra_conf()`), unlike other
`const T& method()` declarations in the codebase. This is likely to be parsed
incorrectly by Cython (and fail to compile).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
