You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/10/07 06:33:01 UTC

[GitHub] [arrow] AlenkaF commented on pull request #14275: ARROW-17160: [C++] Create a base directory for PyArrow CPP header files

AlenkaF commented on PR #14275:
URL: https://github.com/apache/arrow/pull/14275#issuecomment-1271169000

   > There is still the question about moving the include paths from `"arrow/python/.."` to `"pyarrow/.."` (and how to provide some transition path for this), but assuming the current status of exposing those as `"arrow/python/.."` (which seems the safe assumption for 10.0), this seems like a good change to use those paths in the code and headers itself.
   
   I suggest we try to move the include paths from `"arrow/python/.."` to `"pyarrow/.."` for a future release. I can create a separate JIRA issue for it (guessing with Joris we will want to change namespaces from `arrow`&`python` to `pyarrow` also?). Or do you want to tackle this as a part of [ARROW-17838](https://issues.apache.org/jira/browse/ARROW-17838) @kou?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org