[GitHub] [arrow] emkornfield commented on a change in pull request #9504: ARROW-2229: [C++][Python] Add WriteCsv functionality.

2021-02-19 Thread GitBox
emkornfield commented on a change in pull request #9504: URL: https://github.com/apache/arrow/pull/9504#discussion_r577325892 ## File path: cpp/src/arrow/csv/writer.cc ## @@ -0,0 +1,398 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor

[GitHub] [arrow] emkornfield commented on a change in pull request #9504: ARROW-2229: [C++][Python] Add WriteCsv functionality.

2021-02-19 Thread GitBox
emkornfield commented on a change in pull request #9504: URL: https://github.com/apache/arrow/pull/9504#discussion_r577325644 ## File path: cpp/src/arrow/csv/writer.cc ## @@ -0,0 +1,398 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor

[GitHub] [arrow] thamht4190 commented on a change in pull request #8023: ARROW-9318: [C++] Parquet encryption key management

2021-02-19 Thread GitBox
thamht4190 commented on a change in pull request #8023: URL: https://github.com/apache/arrow/pull/8023#discussion_r579596186 ## File path: cpp/src/parquet/encryption/test_encryption_util.cc ## @@ -0,0 +1,481 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [arrow] thamht4190 commented on a change in pull request #8023: ARROW-9318: [C++] Parquet encryption key management

2021-02-19 Thread GitBox
thamht4190 commented on a change in pull request #8023: URL: https://github.com/apache/arrow/pull/8023#discussion_r579596103 ## File path: cpp/src/parquet/encryption/test_encryption_util.cc ## @@ -0,0 +1,481 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [arrow] thamht4190 commented on a change in pull request #8023: ARROW-9318: [C++] Parquet encryption key management

2021-02-19 Thread GitBox
thamht4190 commented on a change in pull request #8023: URL: https://github.com/apache/arrow/pull/8023#discussion_r579596080 ## File path: cpp/src/parquet/encryption/test_encryption_util.cc ## @@ -0,0 +1,481 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [arrow] thamht4190 commented on a change in pull request #8023: ARROW-9318: [C++] Parquet encryption key management

2021-02-19 Thread GitBox
thamht4190 commented on a change in pull request #8023: URL: https://github.com/apache/arrow/pull/8023#discussion_r579595188 ## File path: cpp/src/parquet/encryption/test_encryption_util.cc ## @@ -0,0 +1,481 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782559062 Revision: a5fcad098fe38d434fa3a68baff69adc537b3e68 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782558976 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This is an

[GitHub] [arrow] thamht4190 commented on a change in pull request #8023: ARROW-9318: [C++] Parquet encryption key management

2021-02-19 Thread GitBox
thamht4190 commented on a change in pull request #8023: URL: https://github.com/apache/arrow/pull/8023#discussion_r579594227 ## File path: cpp/src/parquet/encryption/properties_driven_crypto_factory.h ## @@ -0,0 +1,210 @@ +// Licensed to the Apache Software Foundation (ASF)

[GitHub] [arrow] ianmcook removed a comment on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook removed a comment on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782558694 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782558758 Revision: 50822956e907783f82081b5f529dc54b85acd32c Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook removed a comment on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook removed a comment on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782558630 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782558694 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This is an

[GitHub] [arrow] ianmcook edited a comment on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook edited a comment on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782558630 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782558630 @github-actions test-build-vcpkg-win crossbow submit wheel-manylinux2010-cp36m wheel-windows-cp36m This is an

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782557524 Revision: 33a04ae6eff76ec72145ebee381b367fb2596c9b Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782557460 @github-actions crossbow submit wheel-manylinux2010-cp36m wheel-windows-cp36m This is an automated message from

[GitHub] [arrow] ianmcook removed a comment on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook removed a comment on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782556239 @github-actions crossbow submit wheel-manylinux* This is an automated message from the Apache Git

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782556239 @github-actions crossbow submit wheel-manylinux* This is an automated message from the Apache Git Service. To

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782555403 Revision: 6b39c05cc17b6dbb9e0f729f6c2e3e2c4fc16396 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782555329 @github-actions crossbow submit test-build-vcpkg-win wheel-windows-cp36m This is an automated message from the

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782554154 Revision: 6b39c05cc17b6dbb9e0f729f6c2e3e2c4fc16396 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782554060 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782551856 Revision: d315fc6e355e06f15427b466f3bec0fa92e4bee3 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782551713 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782550224 Revision: 8531b648772cc3403313ce1dc7329a2993c2c79f Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] github-actions[bot] commented on pull request #9110: ARROW-11706: [JS] Better BigInt compatibility check

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9110: URL: https://github.com/apache/arrow/pull/9110#issuecomment-782550121 https://issues.apache.org/jira/browse/ARROW-11706 This is an automated message from the Apache Git

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782550095 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] dianaclarke commented on pull request #9110: Better BigInt compatibility check

2021-02-19 Thread GitBox
dianaclarke commented on pull request #9110: URL: https://github.com/apache/arrow/pull/9110#issuecomment-782547787 Thanks for the pull request! I've created a Jira ticket for you: https://issues.apache.org/jira/browse/ARROW-11706 If you could update the PR subject to

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782546116 Revision: 68e83a205fe4743a6a70418aaac1d92e1f2030a3 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782546009 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782543357 Revision: 1f2f34003862e970bd5d90c4cdb150cab492db27 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782542179 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579553038 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579553038 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] westonpace commented on a change in pull request #9533: ARROW-11590: [C++] Move CSV background generator to IO thread pool

2021-02-19 Thread GitBox
westonpace commented on a change in pull request #9533: URL: https://github.com/apache/arrow/pull/9533#discussion_r579564527 ## File path: cpp/src/arrow/util/future.cc ## @@ -250,30 +250,40 @@ class ConcreteFutureImpl : public FutureImpl { } void

[GitHub] [arrow] github-actions[bot] commented on pull request #9533: ARROW-11590: [C++] Move CSV background generator to IO thread pool

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9533: URL: https://github.com/apache/arrow/pull/9533#issuecomment-782497139 https://issues.apache.org/jira/browse/ARROW-11590 This is an automated message from the Apache Git

[GitHub] [arrow] westonpace opened a new pull request #9533: ARROW-11590: [C++] Move CSV background generator to IO thread pool

2021-02-19 Thread GitBox
westonpace opened a new pull request #9533: URL: https://github.com/apache/arrow/pull/9533 The `AsyncThreadedTableReader` was previously creating a dedicated size 1 thread pool for readahead (similar to the way the serial reader behaves). This allowed for parallel readahead because the

[GitHub] [arrow] ursabot edited a comment on pull request #9482: ARROW-11601: [C++][Python][Dataset] expose Parquet pre-buffer option

2021-02-19 Thread GitBox
ursabot edited a comment on pull request #9482: URL: https://github.com/apache/arrow/pull/9482#issuecomment-781568053 Benchmark runs are scheduled for baseline = f1e52b7489a012c2c7d9eae6a3cdde14a52034c2 and contender = 8192b44d2b518d4bfb199c011dfbb0af18af6c7e. Results will be available as

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579554987 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579554987 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579553038 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579554987 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579552853 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,18 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579554987 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579553038 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579553038 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,22 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] dianaclarke commented on a change in pull request #9386: ARROW-11373: [Python][Docs] Add example of specifying type for a column when reading csv file

2021-02-19 Thread GitBox
dianaclarke commented on a change in pull request #9386: URL: https://github.com/apache/arrow/pull/9386#discussion_r579552853 ## File path: docs/source/python/csv.rst ## @@ -75,7 +75,18 @@ Customized conversion - To alter how CSV data is converted to

[GitHub] [arrow] codecov-io commented on pull request #9531: ARROW-11697: [Rust][DataFusion] Add provider for user defined function

2021-02-19 Thread GitBox
codecov-io commented on pull request #9531: URL: https://github.com/apache/arrow/pull/9531#issuecomment-782465926 # [Codecov](https://codecov.io/gh/apache/arrow/pull/9531?src=pr=h1) Report > Merging [#9531](https://codecov.io/gh/apache/arrow/pull/9531?src=pr=desc) (62090ff) into

[GitHub] [arrow] ursabot edited a comment on pull request #9482: ARROW-11601: [C++][Python][Dataset] expose Parquet pre-buffer option

2021-02-19 Thread GitBox
ursabot edited a comment on pull request #9482: URL: https://github.com/apache/arrow/pull/9482#issuecomment-781491306 Benchmark runs are scheduled for baseline = a951c6c74620dd3a9c816c6fa3176b52de80bd34 and contender = 66e9f3d473a1c5701d8e49296fe2d72077cbb236. Results will be available as

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782421207 Revision: 04b6bc993733972b9987138bfe6dc888e7c3357b Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782420952 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782408976 Revision: 87efaf00496f79ec612352c06f22466f1d45e98c Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782407952 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782400291 Revision: 4d0102c84f9826940941eb3da5e206686744d850 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782399190 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782381573 Revision: 667043b46f7c5b2ca77201dcf4123fcae1fd814a Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782380173 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782374008 Revision: 29c90447aef59b5644e0be231c8b8d391d82111d Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782373653 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782370515 Revision: 4047ac6e6d952376ad3ee7123c03716d5692210b Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782370111 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782362760 Revision: bf17f5523da4625018063aa1d009a6bbd560493c Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782362328 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] kou commented on pull request #9294: ARROW-8919: [C++][Compute][Dataset] Add Function::DispatchBest to accomodate implicit casts

2021-02-19 Thread GitBox
kou commented on pull request #9294: URL: https://github.com/apache/arrow/pull/9294#issuecomment-782355055 Thanks! This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782354858 Revision: c53679824d2233621ec7432aae7c4490535a Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782354461 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782348957 Revision: f9109d3f2542c614dc718b8703e45e33a31ec062 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782348397 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782339776 Revision: 3f12299eed86e336d1679c33527aa8db98e62d57 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782339422 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782336580 Revision: 06e17dabfc1e166f9db5c2d3ccfdbff8a148809c Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782336245 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782326040 Revision: 4357a770fa4b3b52e094116a727ef4e0d83e8da4 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782325490 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782310245 Revision: be70cd4b7d60aa86e5a2a87f0a715af2b3068f50 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782309809 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782306103 Revision: 50713dfebb565890f505b43ff1ab9bf6a78d7b9d Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782305744 @github-actions crossbow submit wheel-manylinux2010-cp36m This is an automated message from the Apache Git

[GitHub] [arrow] github-actions[bot] commented on pull request #9532: ARROW-11174: [C++][Dataset] Make expressions available to projection [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9532: URL: https://github.com/apache/arrow/pull/9532#issuecomment-782304056 https://issues.apache.org/jira/browse/ARROW-11174 This is an automated message from the Apache Git

[GitHub] [arrow] bkietz opened a new pull request #9532: ARROW-11174: [C++][Dataset] Make expressions available to projection [WIP]

2021-02-19 Thread GitBox
bkietz opened a new pull request #9532: URL: https://github.com/apache/arrow/pull/9532 [x] Be more explicit about scan time schemas: dataset (aka reader) and projected schemas are independent [x] Ensure field metadata and other attributes are preserved by projection exprs [x]

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782272758 Revision: ea5067dff611a54f5c9788d9a90195fcb6776654 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782272334 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This is an

[GitHub] [arrow] bkietz commented on a change in pull request #9323: ARROW-10438: [C++][Dataset] Partitioning::Format on nulls

2021-02-19 Thread GitBox
bkietz commented on a change in pull request #9323: URL: https://github.com/apache/arrow/pull/9323#discussion_r579306832 ## File path: cpp/src/arrow/compute/kernels/vector_hash.cc ## @@ -529,6 +578,7 @@ std::unique_ptr DictionaryHashInit(KernelContext* ctx,

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782246125 Revision: b78ba7dd6ea941d79e78da259bb5810c96ad5823 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782245606 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This is an

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782220303 Revision: 885fd7364d328ecc3425374b1fb6b3899a66c334 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782219737 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This is an

[GitHub] [arrow] github-actions[bot] commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782207034 Revision: ca2770a255e7e0737fbc1bbf9bd24184495aa1d2 Submitted crossbow builds: [ursacomputing/crossbow @

[GitHub] [arrow] ianmcook commented on pull request #9530: ARROW-11580: [C++] Add CMake option ARROW_DEPENDENCY_SOURCE=VCPKG [WIP]

2021-02-19 Thread GitBox
ianmcook commented on pull request #9530: URL: https://github.com/apache/arrow/pull/9530#issuecomment-782206189 @github-actions crossbow submit test-build-vcpkg-win wheel-manylinux2010-cp36m wheel-windows-cp36m This is an

[GitHub] [arrow] alamb edited a comment on pull request #9531: ARROW-11697: [Rust][DataFusion] Add provider for user defined function

2021-02-19 Thread GitBox
alamb edited a comment on pull request #9531: URL: https://github.com/apache/arrow/pull/9531#issuecomment-782182509 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [arrow] alamb commented on pull request #9531: ARROW-11697: [Rust][DataFusion] Add provider for user defined function

2021-02-19 Thread GitBox
alamb commented on pull request #9531: URL: https://github.com/apache/arrow/pull/9531#issuecomment-782182509 FYI the integration test failures are likely not due to the changes in this PR. The Dev/Lint failures probably need to be looked at. I will review this PR more carefully

[GitHub] [arrow] alamb commented on pull request #9523: ARROW-11687: [Rust][DataFusion] RepartitionExec Hanging

2021-02-19 Thread GitBox
alamb commented on pull request #9523: URL: https://github.com/apache/arrow/pull/9523#issuecomment-782176582 The integration test failed the same way on this unrelated PR as well: https://github.com/apache/arrow/pull/9529/checks?check_run_id=1935707651

[GitHub] [arrow] bkietz closed pull request #8716: ARROW-10655: [C++] Add cache and memoization facility

2021-02-19 Thread GitBox
bkietz closed pull request #8716: URL: https://github.com/apache/arrow/pull/8716 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

[GitHub] [arrow] bkietz commented on pull request #9294: ARROW-8919: [C++][Compute][Dataset] Add Function::DispatchBest to accomodate implicit casts

2021-02-19 Thread GitBox
bkietz commented on pull request #9294: URL: https://github.com/apache/arrow/pull/9294#issuecomment-782163179 @kou indeed, and I have an open PR to fix that https://github.com/apache/arrow/pull/9471 This is an automated

[GitHub] [arrow] bkietz closed pull request #9482: ARROW-11601: [C++][Python][Dataset] expose Parquet pre-buffer option

2021-02-19 Thread GitBox
bkietz closed pull request #9482: URL: https://github.com/apache/arrow/pull/9482 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

[GitHub] [arrow] github-actions[bot] commented on pull request #9531: ARROW-11697: [Rust][DataFusion] Add provider for user defined function

2021-02-19 Thread GitBox
github-actions[bot] commented on pull request #9531: URL: https://github.com/apache/arrow/pull/9531#issuecomment-782093871 https://issues.apache.org/jira/browse/ARROW-11697 This is an automated message from the Apache Git

[GitHub] [arrow] alamb commented on pull request #9454: ARROW-11572: [Rust] Add a kernel for division by single scalar

2021-02-19 Thread GitBox
alamb commented on pull request #9454: URL: https://github.com/apache/arrow/pull/9454#issuecomment-782088630 Makes sense. Thanks @abreis ! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [arrow] abreis commented on pull request #9454: ARROW-11572: [Rust] Add a kernel for division by single scalar

2021-02-19 Thread GitBox
abreis commented on pull request #9454: URL: https://github.com/apache/arrow/pull/9454#issuecomment-782087501 Apologies, I've been swamped this week. I want to add a couple more tests and take a look at Jorge's comments before it's merged. Let me ping you once I do. Thanks!

[GitHub] [arrow] alamb commented on pull request #9509: ARROW-11651: [Rust][DataFusion] Implement Postgres String Functions: Length Functions

2021-02-19 Thread GitBox
alamb commented on pull request #9509: URL: https://github.com/apache/arrow/pull/9509#issuecomment-782086690 The integration test failure appears unrelated https://github.com/apache/arrow/pull/9509/checks?check_run_id=1921983282 I will restart it to try and get a clean run

  1   2   >