Re: [PR] GH-46946: [Python] Raise error if Arrow C++ is built without CSV [arrow]

2025-07-07 Thread via GitHub


conbench-apache-arrow[bot] commented on PR #47008:
URL: https://github.com/apache/arrow/pull/47008#issuecomment-3046172904

   After merging your PR, Conbench analyzed the 4 benchmarking runs that have 
been run so far on merge-commit a0cc2d8ed35dce7ee6c3e7cbcc4867216a9ef16f.
   
   There were 11 benchmark results indicating a performance regression:
   
   - Commit Run on `test-mac-arm` at [2025-07-07 
16:23:15Z](https://conbench.ursa.dev/compare/runs/a419a7cec1ff41babd26efd446fdb72b...a8218fb9e2024d43b9acdbf756c76624/)
 - [`tpch` (R) with engine=arrow, format=native, language=R, 
memory_map=False, query_id=TPCH-10, 
scale_factor=1](https://conbench.ursa.dev/compare/benchmarks/0686be01de6177898000e6b0a09e2d5a...0686bfff4441719f8af9f63747c9)
 - [`tpch` (R) with engine=arrow, format=native, language=R, 
memory_map=False, query_id=TPCH-06, 
scale_factor=1](https://conbench.ursa.dev/compare/benchmarks/0686bdfbee157c85800081f2e34fa32e...0686bff8fc2175928000eb25af58556d)
   - and 9 more (see the report linked below)
   
   The [full Conbench report](https://github.com/apache/arrow/runs/45499937019) 
has more details. It also includes information about 60 possible false 
positives for unstable benchmarks that are known to sometimes produce them.


-- 
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]



Re: [PR] GH-46946: [Python] Raise error if Arrow C++ is built without CSV [arrow]

2025-07-07 Thread via GitHub


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

   Follow-up issue: https://github.com/apache/arrow/issues/47010


-- 
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]



Re: [PR] GH-46946: [Python] Raise error if Arrow C++ is built without CSV [arrow]

2025-07-07 Thread via GitHub


AlenkaF merged PR #47008:
URL: https://github.com/apache/arrow/pull/47008


-- 
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]



Re: [PR] GH-46946: [Python] Raise error if Arrow C++ is built without CSV [arrow]

2025-07-07 Thread via GitHub


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

   Cool, will merge this PR and create a follow-up issue to make CSV optional. 
Thanks!


-- 
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]