[GitHub] [arrow] Dandandan commented on a change in pull request #9776: ARROW-12048: [Rust][DataFusion] Support Common Table Expressions

2021-03-23 Thread GitBox
Dandandan commented on a change in pull request #9776: URL: https://github.com/apache/arrow/pull/9776#discussion_r600216623 ## File path: rust/datafusion/tests/sql.rs ## @@ -1940,6 +1940,56 @@ async fn query_without_from() -> Result<()> { Ok(()) } +#[tokio::test] +async

[GitHub] [arrow] trxcllnt commented on issue #9752: TS Project unable to build with webpack

2021-03-23 Thread GitBox
trxcllnt commented on issue #9752: URL: https://github.com/apache/arrow/issues/9752#issuecomment-805550385 Last I knew, Typescript was against compiling file extensions into import statements. Did this change, or are webpack and typescript at odds now? @westandy does it work if you s

[GitHub] [arrow] Dandandan commented on a change in pull request #9776: ARROW-12048: [Rust][DataFusion] Support Common Table Expressions

2021-03-23 Thread GitBox
Dandandan commented on a change in pull request #9776: URL: https://github.com/apache/arrow/pull/9776#discussion_r600195534 ## File path: rust/datafusion/src/sql/planner.rs ## @@ -103,17 +104,31 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> { /// Generate a logic plan

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359762 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] Dandandan commented on a change in pull request #9776: ARROW-12048: [Rust][DataFusion] Support Common Table Expressions

2021-03-23 Thread GitBox
Dandandan commented on a change in pull request #9776: URL: https://github.com/apache/arrow/pull/9776#discussion_r600195534 ## File path: rust/datafusion/src/sql/planner.rs ## @@ -103,17 +104,31 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> { /// Generate a logic plan

[GitHub] [arrow] github-actions[bot] removed a comment on pull request #9778: ARROW-12052: [Rust] Add Child Data to Arrow's C FFI implementation. …

2021-03-23 Thread GitBox
github-actions[bot] removed a comment on pull request #9778: URL: https://github.com/apache/arrow/pull/9778#issuecomment-804789450 Thanks for opening a pull request! Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW Th

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359888 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] github-actions[bot] commented on pull request #9787: ARROW-12070: [GLib] Drop support for GNU Autotools

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9787: URL: https://github.com/apache/arrow/pull/9787#issuecomment-805515997 Revision: 87bc73ab240671ac09cb4ac50b62d6e73219400c Submitted crossbow builds: [ursacomputing/crossbow @ actions-232](https://github.com/ursacomputing/crossbow/br

[GitHub] [arrow] kou commented on pull request #9787: ARROW-12070: [GLib] Drop support for GNU Autotools

2021-03-23 Thread GitBox
kou commented on pull request #9787: URL: https://github.com/apache/arrow/pull/9787#issuecomment-805515649 @github-actions crossbow submit -g linux -- 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 t

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359877 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359862 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359805 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] emkornfield commented on pull request #9751: ARROW-10606: [C++] Implement Decimal256 casts

2021-03-23 Thread GitBox
emkornfield commented on pull request #9751: URL: https://github.com/apache/arrow/pull/9751#issuecomment-805496501 @lidavidm is it still worth me taking a look or is the rework going to change things substantially? -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [arrow] github-actions[bot] commented on pull request #9787: ARROW-12070: [GLib] Drop support for GNU Autotools

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9787: URL: https://github.com/apache/arrow/pull/9787#issuecomment-805491865 Revision: c2551fd911a60b30f6a20e7868a9b4d4315feee1 Submitted crossbow builds: [ursacomputing/crossbow @ actions-231](https://github.com/ursacomputing/crossbow/br

[GitHub] [arrow] emkornfield commented on pull request #9750: ARROW-11986: [C++][Gandiva] Implement to_char() function on Gandiva

2021-03-23 Thread GitBox
emkornfield commented on pull request #9750: URL: https://github.com/apache/arrow/pull/9750#issuecomment-805490480 The JIRA for this PR looks like it might map to the wrong issue? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [arrow] kou commented on pull request #9787: ARROW-12070: [GLib] Drop support for GNU Autotools

2021-03-23 Thread GitBox
kou commented on pull request #9787: URL: https://github.com/apache/arrow/pull/9787#issuecomment-805486101 @github-actions crossbow submit -g linux -- 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 t

[GitHub] [arrow] emkornfield commented on pull request #9750: ARROW-11986: [C++][Gandiva] Implement to_char() function on Gandiva

2021-03-23 Thread GitBox
emkornfield commented on pull request #9750: URL: https://github.com/apache/arrow/pull/9750#issuecomment-805485975 OK, this PR lacks any documentation to that affect, especially around pattern. Is this meant to emulate sql server or Oracle's to_char? -- This is an automated message from

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359787 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600151615 ## File path: r/tests/testthat/test-compute-sort.R ## @@ -0,0 +1,157 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributo

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600151615 ## File path: r/tests/testthat/test-compute-sort.R ## @@ -0,0 +1,157 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributo

[GitHub] [arrow] emkornfield closed pull request #9489: ARROW-11497: [Python] Provide parquet enable compliant nested type flag for python binding

2021-03-23 Thread GitBox
emkornfield closed pull request #9489: URL: https://github.com/apache/arrow/pull/9489 -- 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. For queries about this service, plea

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600150212 ## File path: r/R/dplyr.R ## @@ -631,17 +671,85 @@ abandon_ship <- function(call, .data, msg = NULL) { eval.parent(call, 2) } -arrange.arrow_dplyr_qu

[GitHub] [arrow] emkornfield commented on pull request #9489: ARROW-11497: [Python] Provide parquet enable compliant nested type flag for python binding

2021-03-23 Thread GitBox
emkornfield commented on pull request #9489: URL: https://github.com/apache/arrow/pull/9489#issuecomment-805472176 +1, going to merge, we can address any items as a followup. @trucnguyenlam apologies for the delay. -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600149524 ## File path: r/R/dplyr.R ## @@ -364,6 +383,7 @@ set_filters <- function(.data, expressions) { collect.arrow_dplyr_query <- function(x, as_data_frame =

[GitHub] [arrow] liyafan82 commented on pull request #9769: ARROW-11899: [Java] Refactor the compression codec implementation into core/Arrow specific parts

2021-03-23 Thread GitBox
liyafan82 commented on pull request #9769: URL: https://github.com/apache/arrow/pull/9769#issuecomment-805469834 > @liyafan82 thanks. A few small comments/nits on documentation, and one question about the API. @emkornfield Thanks for your comments. I will address them ASAP. -- Th

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359762 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] github-actions[bot] commented on pull request #9787: ARROW-12070: [GLib] Drop support for GNU Autotools

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9787: URL: https://github.com/apache/arrow/pull/9787#issuecomment-805461400 https://issues.apache.org/jira/browse/ARROW-12070 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [arrow] kou commented on pull request #9787: ARROW-12070: [GLib] Drop support for GNU Autotools

2021-03-23 Thread GitBox
kou commented on pull request #9787: URL: https://github.com/apache/arrow/pull/9787#issuecomment-805461353 @github-actions crossbow submit -g linux -- 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 t

[GitHub] [arrow] kou opened a new pull request #9787: ARROW-12070: [GLib] Drop support for GNU Autotools

2021-03-23 Thread GitBox
kou opened a new pull request #9787: URL: https://github.com/apache/arrow/pull/9787 Users should use Meson. -- 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. For queries abo

[GitHub] [arrow] emkornfield commented on issue #9737: [Golang] How i create input format for IPC Reader?

2021-03-23 Thread GitBox
emkornfield commented on issue #9737: URL: https://github.com/apache/arrow/issues/9737#issuecomment-805457038 Closing. Please if there are follow-ups let's discuss on the user@ mailing list. One nice addition might be some canonical documentation on how to use the Go API if one doesn't e

[GitHub] [arrow] emkornfield closed issue #9737: [Golang] How i create input format for IPC Reader?

2021-03-23 Thread GitBox
emkornfield closed issue #9737: URL: https://github.com/apache/arrow/issues/9737 -- 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. For queries about this service, please co

[GitHub] [arrow] emkornfield commented on issue #9747: Arrow C++ library not available

2021-03-23 Thread GitBox
emkornfield commented on issue #9747: URL: https://github.com/apache/arrow/issues/9747#issuecomment-805456594 CC @nealrichardson -- 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 com

[GitHub] [arrow] emkornfield commented on issue #9752: TS Project unable to build with webpack

2021-03-23 Thread GitBox
emkornfield commented on issue #9752: URL: https://github.com/apache/arrow/issues/9752#issuecomment-805456446 @westandy typically we like to answer questions on the [user@](https://arrow.apache.org/docs/developers/contributing.html#join-the-mailing-lists) mailing list. I'm going to CC som

[GitHub] [arrow] emkornfield commented on a change in pull request #9769: ARROW-11899: [Java] Refactor the compression codec implementation into core/Arrow specific parts

2021-03-23 Thread GitBox
emkornfield commented on a change in pull request #9769: URL: https://github.com/apache/arrow/pull/9769#discussion_r600131274 ## File path: java/compression/src/main/java/org/apache/arrow/compression/Lz4CompressionCodec.java ## @@ -38,109 +36,39 @@ /** * Compression codec f

[GitHub] [arrow] westonpace commented on pull request #9643: ARROW-11883: [C++] Add ConcatMap, MergeMap, and an async-reentrant version of Map

2021-03-23 Thread GitBox
westonpace commented on pull request #9643: URL: https://github.com/apache/arrow/pull/9643#issuecomment-805454734 Assuming remaining jobs pass (Travis looks like it is backlogged ~12 hours) or are unrelated, please merge. -- This is an automated message from the Apache Git Service. To re

[GitHub] [arrow] emkornfield commented on a change in pull request #9769: ARROW-11899: [Java] Refactor the compression codec implementation into core/Arrow specific parts

2021-03-23 Thread GitBox
emkornfield commented on a change in pull request #9769: URL: https://github.com/apache/arrow/pull/9769#discussion_r600129388 ## File path: java/vector/src/main/java/org/apache/arrow/vector/compression/AbstractCompressionCodec.java ## @@ -0,0 +1,117 @@ +/* + * Licensed to the

[GitHub] [arrow] emkornfield closed issue #9781: Reading a parquet with timestamp out of bounds in pandas

2021-03-23 Thread GitBox
emkornfield closed issue #9781: URL: https://github.com/apache/arrow/issues/9781 -- 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. For queries about this service, please co

[GitHub] [arrow] emkornfield commented on issue #9781: Reading a parquet with timestamp out of bounds in pandas

2021-03-23 Thread GitBox
emkornfield commented on issue #9781: URL: https://github.com/apache/arrow/issues/9781#issuecomment-805451756 Hi @guidocioni, You could probably read the data into an [arrow table](https://arrow.apache.org/docs/python/generated/pyarrow.parquet.read_table.html?highlight=read_table#pyarrow

[GitHub] [arrow] seddonm1 commented on pull request #9428: ARROW-10354: [Rust][DataFusion] regexp_extract function to select regex groups from strings

2021-03-23 Thread GitBox
seddonm1 commented on pull request #9428: URL: https://github.com/apache/arrow/pull/9428#issuecomment-805451302 @sweb yes, i will have a look in the next day or two -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600118690 ## File path: r/R/arrow-datum.R ## @@ -138,3 +138,16 @@ as.integer.ArrowDatum <- function(x, ...) as.integer(as.vector(x), ...) #' @export as.characte

[GitHub] [arrow] liyafan82 closed pull request #9744: ARROW-12012: [Java][JDBC] Fix BinaryConsumer reallocation

2021-03-23 Thread GitBox
liyafan82 closed pull request #9744: URL: https://github.com/apache/arrow/pull/9744 -- 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. For queries about this service, please

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600111676 ## File path: r/R/dplyr.R ## @@ -631,17 +671,85 @@ abandon_ship <- function(call, .data, msg = NULL) { eval.parent(call, 2) } -arrange.arrow_dplyr_qu

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600110895 ## File path: r/R/dplyr.R ## @@ -631,17 +671,85 @@ abandon_ship <- function(call, .data, msg = NULL) { eval.parent(call, 2) } -arrange.arrow_dplyr_qu

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600110657 ## File path: r/R/dplyr.R ## @@ -631,17 +671,85 @@ abandon_ship <- function(call, .data, msg = NULL) { eval.parent(call, 2) } -arrange.arrow_dplyr_qu

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600109663 ## File path: r/R/dplyr.R ## @@ -390,6 +414,16 @@ collect.arrow_dplyr_query <- function(x, as_data_frame = TRUE, ...) { tab <- RecordBatch$create

[GitHub] [arrow] liyafan82 commented on pull request #9744: ARROW-12012: [Java][JDBC] Fix BinaryConsumer reallocation

2021-03-23 Thread GitBox
liyafan82 commented on pull request #9744: URL: https://github.com/apache/arrow/pull/9744#issuecomment-805416730 +1. Thanks @zxf -- 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 com

[GitHub] [arrow] github-actions[bot] commented on pull request #9786: ARROW-11693: [C++] Add string length kernel

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9786: URL: https://github.com/apache/arrow/pull/9786#issuecomment-805412779 https://issues.apache.org/jira/browse/ARROW-11693 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [arrow] github-actions[bot] commented on pull request #9786: ARROW-11693: [C++] Add string length kernel

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9786: URL: https://github.com/apache/arrow/pull/9786#issuecomment-805400529 Thanks for opening a pull request! Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW Then could

[GitHub] [arrow] github-actions[bot] commented on pull request #9773: ARROW-12028 ARROW-11940: [Rust][DataFusion] Add TimestampMillisecond support to GROUP BY/hash aggregates

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9773: URL: https://github.com/apache/arrow/pull/9773#issuecomment-805400295 https://issues.apache.org/jira/browse/ARROW-12028 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359862 -- 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. For qu

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359805 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359862 -- 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. For qu

[GitHub] [arrow] kou commented on a change in pull request #9767: [Packaging] Use vcpkg to build macOS wheels

2021-03-23 Thread GitBox
kou commented on a change in pull request #9767: URL: https://github.com/apache/arrow/pull/9767#discussion_r600063752 ## File path: dev/tasks/crossbow.py ## @@ -322,7 +322,11 @@ def create_tree(self, files): def create_commit(self, files, parents=None, message='',

[GitHub] [arrow] edponce opened a new pull request #9786: ARROW 11693: [C++] dd string length kernel

2021-03-23 Thread GitBox
edponce opened a new pull request #9786: URL: https://github.com/apache/arrow/pull/9786 -- 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. For queries about this service, pl

[GitHub] [arrow] kou commented on a change in pull request #9767: [Packaging] Use vcpkg to build macOS wheels

2021-03-23 Thread GitBox
kou commented on a change in pull request #9767: URL: https://github.com/apache/arrow/pull/9767#discussion_r600062036 ## File path: dev/tasks/python-wheels/github.osx.yml ## @@ -31,88 +43,107 @@ jobs: - name: Checkout Arrow run: | set -ex -

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359787 -- 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. For qu

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r600060349 ## File path: r/R/dplyr.R ## @@ -364,6 +383,7 @@ set_filters <- function(.data, expressions) { collect.arrow_dplyr_query <- function(x, as_data_frame =

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359787 Benchmark runs are scheduled for baseline = ee24399c52c4fb704cbb2a69d489792505bece53 and contender = efcd862d87912e4efd93da3dc124d3b31d5d3e6b. Results will be available as

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359762 -- 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. For qu

[GitHub] [arrow] ursabot edited a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot edited a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359762 -- 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. For qu

[GitHub] [arrow] ursabot commented on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot commented on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359862 -- 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. For queries a

[GitHub] [arrow] ursabot commented on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot commented on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-805359762 -- 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. For queries a

[GitHub] [arrow] nealrichardson closed pull request #9757: ARROW-12029: [R] Remove args from FeatherReader$create v2

2021-03-23 Thread GitBox
nealrichardson closed pull request #9757: URL: https://github.com/apache/arrow/pull/9757 -- 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. For queries about this service, p

[GitHub] [arrow] github-actions[bot] commented on pull request #9785: ARROW-12069: [C++][Gandiva] Implement IN expressions for Decimal type

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9785: URL: https://github.com/apache/arrow/pull/9785#issuecomment-805342020 https://issues.apache.org/jira/browse/ARROW-12069 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [arrow] velvia commented on a change in pull request #9773: [rust][datafusion] Fix 12028 and 11940: Add TimestampMillisecond support to GROUP BY/hash aggregates

2021-03-23 Thread GitBox
velvia commented on a change in pull request #9773: URL: https://github.com/apache/arrow/pull/9773#discussion_r600023805 ## File path: rust/datafusion/src/physical_plan/hash_join.rs ## @@ -656,42 +656,20 @@ fn equal_rows( .zip(right_arrays) .all(|(l, r)| match

[GitHub] [arrow] velvia commented on a change in pull request #9773: [rust][datafusion] Fix 12028 and 11940: Add TimestampMillisecond support to GROUP BY/hash aggregates

2021-03-23 Thread GitBox
velvia commented on a change in pull request #9773: URL: https://github.com/apache/arrow/pull/9773#discussion_r600023253 ## File path: rust/datafusion/src/scalar.rs ## @@ -19,6 +19,11 @@ use std::{convert::TryFrom, fmt, iter::repeat, sync::Arc}; +use arrow::array::{ Revie

[GitHub] [arrow] westonpace commented on a change in pull request #9779: ARROW-12056: [C++] Create sequencing AsyncGenerator

2021-03-23 Thread GitBox
westonpace commented on a change in pull request #9779: URL: https://github.com/apache/arrow/pull/9779#discussion_r67226 ## File path: cpp/src/arrow/util/async_generator.h ## @@ -278,6 +278,147 @@ AsyncGenerator MakeMappedGenerator(AsyncGenerator source_generator, retur

[GitHub] [arrow] ursabot removed a comment on pull request #9272: [WIP] Benchmark placebo

2021-03-23 Thread GitBox
ursabot removed a comment on pull request #9272: URL: https://github.com/apache/arrow/pull/9272#issuecomment-804609269 -- 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. For q

[GitHub] [arrow] Dandandan commented on pull request #9773: [rust][datafusion] Fix 12028 and 11940: Add TimestampMillisecond support to GROUP BY/hash aggregates

2021-03-23 Thread GitBox
Dandandan commented on pull request #9773: URL: https://github.com/apache/arrow/pull/9773#issuecomment-805313128 This looks good @velvia , I added some minor comments. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [arrow] Dandandan commented on a change in pull request #9773: [rust][datafusion] Fix 12028 and 11940: Add TimestampMillisecond support to GROUP BY/hash aggregates

2021-03-23 Thread GitBox
Dandandan commented on a change in pull request #9773: URL: https://github.com/apache/arrow/pull/9773#discussion_r64193 ## File path: rust/datafusion/src/physical_plan/hash_join.rs ## @@ -656,42 +656,20 @@ fn equal_rows( .zip(right_arrays) .all(|(l, r)| ma

[GitHub] [arrow] Dandandan commented on a change in pull request #9773: [rust][datafusion] Fix 12028 and 11940: Add TimestampMillisecond support to GROUP BY/hash aggregates

2021-03-23 Thread GitBox
Dandandan commented on a change in pull request #9773: URL: https://github.com/apache/arrow/pull/9773#discussion_r63841 ## File path: rust/datafusion/src/scalar.rs ## @@ -19,6 +19,11 @@ use std::{convert::TryFrom, fmt, iter::repeat, sync::Arc}; +use arrow::array::{ Re

[GitHub] [arrow] Dandandan commented on a change in pull request #9776: ARROW-12048: [Rust][DataFusion] Support Common Table Expressions

2021-03-23 Thread GitBox
Dandandan commented on a change in pull request #9776: URL: https://github.com/apache/arrow/pull/9776#discussion_r61227 ## File path: rust/datafusion/tests/sql.rs ## @@ -1940,6 +1940,56 @@ async fn query_without_from() -> Result<()> { Ok(()) } +#[tokio::test] +async

[GitHub] [arrow] nealrichardson commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
nealrichardson commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r599985192 ## File path: r/R/arrow-datum.R ## @@ -138,3 +138,16 @@ as.integer.ArrowDatum <- function(x, ...) as.integer(as.vector(x), ...) #' @export as.ch

[GitHub] [arrow] alamb commented on pull request #9762: ARROW-12037: [Rust] [DataFusion] Support catalogs and schemas for table namespacing

2021-03-23 Thread GitBox
alamb commented on pull request #9762: URL: https://github.com/apache/arrow/pull/9762#issuecomment-805304397 Thanks again @returnString ! This is a great feature -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [arrow] alamb closed pull request #9762: ARROW-12037: [Rust] [DataFusion] Support catalogs and schemas for table namespacing

2021-03-23 Thread GitBox
alamb closed pull request #9762: URL: https://github.com/apache/arrow/pull/9762 -- 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. For queries about this service, please con

[GitHub] [arrow] alamb commented on pull request #9773: [rust][datafusion] Fix 12028 and 11940: Add TimestampMillisecond support to GROUP BY/hash aggregates

2021-03-23 Thread GitBox
alamb commented on pull request #9773: URL: https://github.com/apache/arrow/pull/9773#issuecomment-805303369 Thanks @velvia -- I plan to review this tomorrow -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [arrow] alamb commented on pull request #9776: ARROW-12048: [Rust][DataFusion] Support Common Table Expressions

2021-03-23 Thread GitBox
alamb commented on pull request #9776: URL: https://github.com/apache/arrow/pull/9776#issuecomment-805303051 Thank you for this @Dandandan ! I plan to review this carefully tomorrow -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [arrow] jorgecarleitao commented on a change in pull request #9776: ARROW-12048: [Rust][DataFusion] Support Common Table Expressions

2021-03-23 Thread GitBox
jorgecarleitao commented on a change in pull request #9776: URL: https://github.com/apache/arrow/pull/9776#discussion_r54379 ## File path: rust/datafusion/tests/sql.rs ## @@ -1940,6 +1940,56 @@ async fn query_without_from() -> Result<()> { Ok(()) } +#[tokio::test] +

[GitHub] [arrow] rodrigojdebem commented on pull request #9750: ARROW-11986: [C++][Gandiva] Implement to_char() function on Gandiva

2021-03-23 Thread GitBox
rodrigojdebem commented on pull request #9750: URL: https://github.com/apache/arrow/pull/9750#issuecomment-805294095 > Is this a duplicate of: https://github.com/apache/arrow/pull/8158/files? @emkornfield the idea of `to_char` is to allow a pattern to be passed as well so the number

[GitHub] [arrow] kszucs commented on a change in pull request #9767: [Packaging] Use vcpkg to build macOS wheels

2021-03-23 Thread GitBox
kszucs commented on a change in pull request #9767: URL: https://github.com/apache/arrow/pull/9767#discussion_r599985796 ## File path: dev/tasks/crossbow.py ## @@ -322,7 +322,11 @@ def create_tree(self, files): def create_commit(self, files, parents=None, message='',

[GitHub] [arrow] kszucs commented on a change in pull request #9767: [Packaging] Use vcpkg to build macOS wheels

2021-03-23 Thread GitBox
kszucs commented on a change in pull request #9767: URL: https://github.com/apache/arrow/pull/9767#discussion_r599982868 ## File path: dev/tasks/crossbow.py ## @@ -322,7 +322,11 @@ def create_tree(self, files): def create_commit(self, files, parents=None, message='',

[GitHub] [arrow] kszucs commented on a change in pull request #9767: [Packaging] Use vcpkg to build macOS wheels

2021-03-23 Thread GitBox
kszucs commented on a change in pull request #9767: URL: https://github.com/apache/arrow/pull/9767#discussion_r599982093 ## File path: dev/tasks/python-wheels/github.osx.yml ## @@ -31,88 +43,107 @@ jobs: - name: Checkout Arrow run: | set -ex -

[GitHub] [arrow] kszucs commented on a change in pull request #9767: [Packaging] Use vcpkg to build macOS wheels

2021-03-23 Thread GitBox
kszucs commented on a change in pull request #9767: URL: https://github.com/apache/arrow/pull/9767#discussion_r599981916 ## File path: dev/tasks/python-wheels/github.osx.yml ## @@ -31,88 +43,107 @@ jobs: - name: Checkout Arrow run: | set -ex -

[GitHub] [arrow] frank400 opened a new pull request #9785: ARROW-12069: [C++][Gandiva] Implement IN expressions for Decimal type

2021-03-23 Thread GitBox
frank400 opened a new pull request #9785: URL: https://github.com/apache/arrow/pull/9785 Implement support to IN expressions at Arrow's decimals field independently of its precision or scale. -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [arrow] kou commented on a change in pull request #9767: [Packaging] Use vcpkg to build macOS wheels

2021-03-23 Thread GitBox
kou commented on a change in pull request #9767: URL: https://github.com/apache/arrow/pull/9767#discussion_r599966767 ## File path: ci/scripts/python_wheel_macos_build.sh ## @@ -0,0 +1,135 @@ +#!/usr/bin/env bash +# +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r599969084 ## File path: r/R/dplyr.R ## @@ -631,17 +669,98 @@ abandon_ship <- function(call, .data, msg = NULL) { eval.parent(call, 2) } -arrange.arrow_dplyr_qu

[GitHub] [arrow] nealrichardson commented on a change in pull request #9757: ARROW-12029: [R] Remove args from FeatherReader$create v2

2021-03-23 Thread GitBox
nealrichardson commented on a change in pull request #9757: URL: https://github.com/apache/arrow/pull/9757#discussion_r599964739 ## File path: r/R/feather.R ## @@ -178,8 +178,6 @@ read_feather <- function(file, col_select = NULL, as_data_frame = TRUE, ...) { #' takes the foll

[GitHub] [arrow] kou closed pull request #9777: ARROW-12051: [GLib] Keep input stream reference of GArrowCSVReader

2021-03-23 Thread GitBox
kou closed pull request #9777: URL: https://github.com/apache/arrow/pull/9777 -- 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. For queries about this service, please conta

[GitHub] [arrow] kou commented on pull request #9777: ARROW-12051: [GLib] Keep input stream reference of GArrowCSVReader

2021-03-23 Thread GitBox
kou commented on pull request #9777: URL: https://github.com/apache/arrow/pull/9777#issuecomment-805251400 +1 -- 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. For queries a

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r599917166 ## File path: r/R/dplyr.R ## @@ -390,6 +409,25 @@ collect.arrow_dplyr_query <- function(x, as_data_frame = TRUE, ...) { tab <- RecordBatch$create

[GitHub] [arrow] github-actions[bot] commented on pull request #9784: ARROW-12067: [Python][Doc] Document pyarrow_(un)wrap_scalar

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9784: URL: https://github.com/apache/arrow/pull/9784#issuecomment-805223986 https://issues.apache.org/jira/browse/ARROW-12067 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r599912716 ## File path: r/R/dplyr.R ## @@ -631,17 +669,98 @@ abandon_ship <- function(call, .data, msg = NULL) { eval.parent(call, 2) } -arrange.arrow_dplyr_qu

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r599912271 ## File path: r/R/expression.R ## @@ -181,6 +181,20 @@ find_array_refs <- function(x) { unlist(out) } +get_field_name <- function(x, msg = NULL) {

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r599912007 ## File path: r/R/dplyr.R ## @@ -390,6 +409,25 @@ collect.arrow_dplyr_query <- function(x, as_data_frame = TRUE, ...) { tab <- RecordBatch$create

[GitHub] [arrow] ianmcook commented on a change in pull request #9745: ARROW-11703: [R] Implement dplyr::arrange() [WIP]

2021-03-23 Thread GitBox
ianmcook commented on a change in pull request #9745: URL: https://github.com/apache/arrow/pull/9745#discussion_r599911766 ## File path: r/R/array.R ## @@ -131,6 +133,12 @@ Array <- R6Class("Array", assert_is(i, "Array") call_function("filter", self, i, options =

[GitHub] [arrow] lidavidm commented on a change in pull request #9779: ARROW-12056: [C++] Create sequencing AsyncGenerator

2021-03-23 Thread GitBox
lidavidm commented on a change in pull request #9779: URL: https://github.com/apache/arrow/pull/9779#discussion_r599902529 ## File path: cpp/src/arrow/util/async_generator.h ## @@ -278,6 +278,147 @@ AsyncGenerator MakeMappedGenerator(AsyncGenerator source_generator, return

[GitHub] [arrow] bkietz closed pull request #9621: ARROW-11591: [C++][Compute] Grouped aggregation

2021-03-23 Thread GitBox
bkietz closed pull request #9621: URL: https://github.com/apache/arrow/pull/9621 -- 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. For queries about this service, please co

[GitHub] [arrow] bkietz commented on pull request #9621: ARROW-11591: [C++][Compute] Grouped aggregation

2021-03-23 Thread GitBox
bkietz commented on pull request #9621: URL: https://github.com/apache/arrow/pull/9621#issuecomment-805201203 +1, merging -- 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. F

[GitHub] [arrow] westonpace commented on a change in pull request #9643: ARROW-11883: [C++] Add ConcatMap, MergeMap, and an async-reentrant version of Map

2021-03-23 Thread GitBox
westonpace commented on a change in pull request #9643: URL: https://github.com/apache/arrow/pull/9643#discussion_r599887016 ## File path: cpp/src/arrow/util/vector.h ## @@ -81,5 +84,53 @@ std::vector FilterVector(std::vector values, Predicate&& predicate) { return values;

[GitHub] [arrow] github-actions[bot] commented on pull request #9783: ARROW-12065: [C++][Python] Fix segfault reading JSON file

2021-03-23 Thread GitBox
github-actions[bot] commented on pull request #9783: URL: https://github.com/apache/arrow/pull/9783#issuecomment-805190245 https://issues.apache.org/jira/browse/ARROW-12065 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

  1   2   >