[GitHub] [arrow] jorisvandenbossche commented on a diff in pull request #14781: GH-14778: [Python] Add (Chunked)Array sort() method

2022-12-01 Thread GitBox
jorisvandenbossche commented on code in PR #14781: URL: https://github.com/apache/arrow/pull/14781#discussion_r1037873759 ## python/pyarrow/array.pxi: ## @@ -2743,6 +2766,36 @@ cdef class StructArray(Array): result.validate() return result +def sort(self,

[GitHub] [arrow] wgtmac commented on a diff in pull request #14191: ARROW-17798: [C++][Parquet] Add DELTA_BINARY_PACKED encoder to Parquet writer

2022-12-01 Thread GitBox
wgtmac commented on code in PR #14191: URL: https://github.com/apache/arrow/pull/14191#discussion_r1037821973 ## cpp/src/arrow/util/bit_stream_utils.h: ## @@ -204,8 +204,10 @@ class BitReader { inline bool BitWriter::PutValue(uint64_t v, int num_bits) { // TODO: revisit th

[GitHub] [arrow] jorisvandenbossche commented on issue #14726: pq.read_table("parquet files path", memory_map=True) still consume large memory space(200G file cost 200G memory and slow)

2022-12-01 Thread GitBox
jorisvandenbossche commented on issue #14726: URL: https://github.com/apache/arrow/issues/14726#issuecomment-1334867906 > It might be nice to have a higher-level function to read a Parquet file as a stream of batches by the way. There are already two options, I think: - With t

[GitHub] [arrow] github-actions[bot] commented on pull request #14235: ARROW-17692: [R] Add support for building with system AWS SDK C++

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14235: URL: https://github.com/apache/arrow/pull/14235#issuecomment-1334857675 Revision: 95fbfd6878428720193b600a64ad263d18f7d6f5 Submitted crossbow builds: [ursacomputing/crossbow @ actions-6ec57f27eb](https://github.com/ursacomputing/crossbow/bra

[GitHub] [arrow] kou commented on pull request #14235: ARROW-17692: [R] Add support for building with system AWS SDK C++

2022-12-01 Thread GitBox
kou commented on PR #14235: URL: https://github.com/apache/arrow/pull/14235#issuecomment-1334853415 @github-actions crossbow submit -g r -- 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 specif

[GitHub] [arrow-datafusion] Ted-Jiang commented on issue #361: Support window functions with window frame

2022-12-01 Thread GitBox
Ted-Jiang commented on issue #361: URL: https://github.com/apache/arrow-datafusion/issues/361#issuecomment-1334838512 @ozankabak Thanks again for supporting window frame.👍 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [arrow-datafusion] Ted-Jiang commented on issue #361: Support window functions with window frame

2022-12-01 Thread GitBox
Ted-Jiang commented on issue #361: URL: https://github.com/apache/arrow-datafusion/issues/361#issuecomment-1334837480 I have run some query success in out internal version ballista 😄 and fix a lot issue -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [arrow-datafusion] Ted-Jiang commented on issue #361: Support window functions with window frame

2022-12-01 Thread GitBox
Ted-Jiang commented on issue #361: URL: https://github.com/apache/arrow-datafusion/issues/361#issuecomment-1334833899 i am going to fix this `window func` issue. So we can do `window func` in ballista ✌️ -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [arrow] ursabot commented on pull request #14811: GH-14809: [Dev] Add created GitHub issues to iss...@arrow.apache.org

2022-12-01 Thread GitBox
ursabot commented on PR #14811: URL: https://github.com/apache/arrow/pull/14811#issuecomment-1334824437 Benchmark runs are scheduled for baseline = 0db0b1f37bf192f9aacfe0d8faf0c178d00c6238 and contender = 7f04d919316e825ad0978bdf9a26c19769cd13b5. 7f04d919316e825ad0978bdf9a26c19769cd13b5 is

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037829130 ## cpp/src/parquet/page_index.cc: ## @@ -0,0 +1,254 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. S

[GitHub] [arrow] github-actions[bot] commented on pull request #14821: Bump decode-uri-component from 0.2.0 to 0.2.2 in /js

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14821: URL: https://github.com/apache/arrow/pull/14821#issuecomment-1334822599 Thanks for opening a pull request! If this is not a [minor PR](https://github.com/apache/arrow/blob/master/CONTRIBUTING.md#Minor-Fixes). Could you open an issue

[GitHub] [arrow] dependabot[bot] opened a new pull request, #14821: Bump decode-uri-component from 0.2.0 to 0.2.2 in /js

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #14821: URL: https://github.com/apache/arrow/pull/14821 Bumps [decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) from 0.2.0 to 0.2.2. Release notes Sourced from https://github.com/SamVerschueren/decode-uri-comp

[GitHub] [arrow] github-actions[bot] commented on pull request #14820: GH-14819: [CI][RPM] Add workaround for build failure on CentOS 9 Stream

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14820: URL: https://github.com/apache/arrow/pull/14820#issuecomment-1334822272 Revision: a9b21cf73528becd04f8623e3842a5cced6b7a53 Submitted crossbow builds: [ursacomputing/crossbow @ actions-0345cd5665](https://github.com/ursacomputing/crossbow/bra

[GitHub] [arrow] kou commented on issue #14784: [Dev] Allow users to assign issues with a comment on the issue

2022-12-01 Thread GitBox
kou commented on issue #14784: URL: https://github.com/apache/arrow/issues/14784#issuecomment-1334822134 Issue resolved by pull request 14785 https://github.com/apache/arrow/pull/14785 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [arrow] kou merged pull request #14785: GH-14784: [Dev] Add possibility to autoassign on GitHub issue comment

2022-12-01 Thread GitBox
kou merged PR #14785: URL: https://github.com/apache/arrow/pull/14785 -- 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-unsubscr...@arrow.apache.or

[GitHub] [arrow] github-actions[bot] commented on pull request #14820: GH-14819: [CI][RPM] Add workaround for build failure on CentOS 9 Stream

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14820: URL: https://github.com/apache/arrow/pull/14820#issuecomment-1334820835 * Closes: #14819 -- 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 c

[GitHub] [arrow] kou commented on pull request #14820: GH-14819: [CI][RPM] Add workaround for build failure on CentOS 9 Stream

2022-12-01 Thread GitBox
kou commented on PR #14820: URL: https://github.com/apache/arrow/pull/14820#issuecomment-1334820959 @github-actions crossbow submit centos-9-* -- 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

[GitHub] [arrow] github-actions[bot] commented on pull request #14820: GH-14819: [CI][RPM] Add workaround for build failure on CentOS 9 Stream

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14820: URL: https://github.com/apache/arrow/pull/14820#issuecomment-1334820858 :warning: GitHub issue #14819 **has been automatically assigned in GitHub** to PR creator. -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [arrow] kou opened a new pull request, #14820: GH-14819: [CI][RPM] Add workaround for build failure on CentOS 9 Stream

2022-12-01 Thread GitBox
kou opened a new pull request, #14820: URL: https://github.com/apache/arrow/pull/14820 Closes: #14819 https://github.com/ursacomputing/crossbow/actions/runs/3590866755/jobs/6044735050#step:6:1864 CMake Error at /usr/share/cmake/Modules/CMakeTestCCompiler.cmake:66 (message)

[GitHub] [arrow] github-actions[bot] commented on pull request #14820: GH-14819: [CI][RPM] Add workaround for build failure on CentOS 9 Stream

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14820: URL: https://github.com/apache/arrow/pull/14820#issuecomment-1334820871 :warning: GitHub issue #14819 **has no components**, please add labels for components. -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037824655 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,137 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. Se

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037824100 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,137 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. Se

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037822389 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,137 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. Se

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037822297 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,137 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. Se

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037822111 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,137 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. Se

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037821275 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,137 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. Se

[GitHub] [arrow] emkornfield commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
emkornfield commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037821000 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,137 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. Se

[GitHub] [arrow] wgtmac commented on a diff in pull request #14806: ARROW-18421: [C++][ORC] Add accessor for number of rows by stripe in reader

2022-12-01 Thread GitBox
wgtmac commented on code in PR #14806: URL: https://github.com/apache/arrow/pull/14806#discussion_r1037818902 ## cpp/src/arrow/adapters/orc/adapter.cc: ## @@ -217,6 +217,8 @@ class ORCFileReader::Impl { int64_t NumberOfRows() { return static_cast(reader_->getNumberOfRows()

[GitHub] [arrow] kou commented on a diff in pull request #14235: ARROW-17692: [R] Add support for building with system AWS SDK C++

2022-12-01 Thread GitBox
kou commented on code in PR #14235: URL: https://github.com/apache/arrow/pull/14235#discussion_r1037803999 ## dev/tasks/r/github.macos-linux.local.yml: ## @@ -45,11 +45,20 @@ jobs: - name: Configure non-autobrew dependencies (linux) run: | sudo apt-get

[GitHub] [arrow] github-actions[bot] commented on pull request #14235: ARROW-17692: [R] Add support for building with system AWS SDK C++

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14235: URL: https://github.com/apache/arrow/pull/14235#issuecomment-1334787468 Revision: e40a6374ab8fbecf9904aebf0545f703d748c160 Submitted crossbow builds: [ursacomputing/crossbow @ actions-28b92fc3ac](https://github.com/ursacomputing/crossbow/bra

[GitHub] [arrow] kou commented on pull request #14235: ARROW-17692: [R] Add support for building with system AWS SDK C++

2022-12-01 Thread GitBox
kou commented on PR #14235: URL: https://github.com/apache/arrow/pull/14235#issuecomment-1334785087 @github-actions crossbow submit -g r -- 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 specif

[GitHub] [arrow-datafusion] mingmwang commented on pull request #4447: fix `push_down_filter` push `Expr` instead of `column`.

2022-12-01 Thread GitBox
mingmwang commented on PR #4447: URL: https://github.com/apache/arrow-datafusion/pull/4447#issuecomment-1334781946 LGTM. -- 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. T

[GitHub] [arrow] github-actions[bot] commented on pull request #14818: ARROW-17374: [C++] Snappy package may be built without CMAKE_BUILD_TYPE

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14818: URL: https://github.com/apache/arrow/pull/14818#issuecomment-1334771625 :warning: Ticket **has not been started in JIRA**, please click 'Start Progress'. -- This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [arrow] github-actions[bot] commented on pull request #14818: ARROW-17374: [C++] Snappy package may be built without CMAKE_BUILD_TYPE

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14818: URL: https://github.com/apache/arrow/pull/14818#issuecomment-1334771605 https://issues.apache.org/jira/browse/ARROW-17374 -- 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] kou opened a new pull request, #14818: ARROW-17374: [C++] Snappy package may be built without CMAKE_BUILD_TYPE

2022-12-01 Thread GitBox
kou opened a new pull request, #14818: URL: https://github.com/apache/arrow/pull/14818 For example, conda's Snappy package is built without CMAKE_BUILD_TYPE. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [arrow] ursabot commented on pull request #14808: GH-14805: [Format] C Data Interface: clarify nullability of buffer pointers

2022-12-01 Thread GitBox
ursabot commented on PR #14808: URL: https://github.com/apache/arrow/pull/14808#issuecomment-1334743666 Benchmark runs are scheduled for baseline = 02bbcc5057302696227dc4a7be004c043ad7a68e and contender = 0db0b1f37bf192f9aacfe0d8faf0c178d00c6238. 0db0b1f37bf192f9aacfe0d8faf0c178d00c6238 is

[GitHub] [arrow-datafusion] liukun4515 closed pull request #4473: minor fix: doc format

2022-12-01 Thread GitBox
liukun4515 closed pull request #4473: minor fix: doc format URL: https://github.com/apache/arrow-datafusion/pull/4473 -- 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 unsubsc

[GitHub] [arrow-datafusion] xudong963 commented on a diff in pull request #4472: Drive sqllogictest runner on directory contents rather than hard coded list

2022-12-01 Thread GitBox
xudong963 commented on code in PR #4472: URL: https://github.com/apache/arrow-datafusion/pull/4472#discussion_r1037748963 ## datafusion/core/tests/sql/information_schema.rs: ## @@ -669,98 +669,3 @@ async fn show_external_create_table() { async fn plan_and_collect(ctx: &SessionC

[GitHub] [arrow-datafusion] xudong963 commented on a diff in pull request #4469: MINOR: Add note about sqllogictest to contributor guide

2022-12-01 Thread GitBox
xudong963 commented on code in PR #4469: URL: https://github.com/apache/arrow-datafusion/pull/4469#discussion_r1037761511 ## docs/source/contributor-guide/index.md: ## @@ -134,6 +134,10 @@ cargo test -p datafusion --tests sql_integration One very important test is the [sql_i

[GitHub] [arrow-datafusion] ursabot commented on pull request #4425: Fix `push down limit` regression when meet SubqueryAlias

2022-12-01 Thread GitBox
ursabot commented on PR #4425: URL: https://github.com/apache/arrow-datafusion/pull/4425#issuecomment-1334714580 Benchmark runs are scheduled for baseline = fb8eeb2ac80adeb872a32fc95cb29dbc17f0de60 and contender = 1aa645f8d06a751c6e3dcc86b7e1f525bfcc1d9f. 1aa645f8d06a751c6e3dcc86b7e1f525b

[GitHub] [arrow-datafusion] liukun4515 merged pull request #4425: Fix `push down limit` regression when meet SubqueryAlias

2022-12-01 Thread GitBox
liukun4515 merged PR #4425: URL: https://github.com/apache/arrow-datafusion/pull/4425 -- 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-unsubscr...

[GitHub] [arrow-datafusion] liukun4515 opened a new pull request, #4473: minor fix: doc format

2022-12-01 Thread GitBox
liukun4515 opened a new pull request, #4473: URL: https://github.com/apache/arrow-datafusion/pull/4473 # Which issue does this PR close? Closes #. # Rationale for this change # What changes are included in this PR? # Are these changes tested

[GitHub] [arrow] wgtmac commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
wgtmac commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037745297 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,143 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. See the

[GitHub] [arrow] wgtmac commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
wgtmac commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037745138 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,143 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. See the

[GitHub] [arrow-datafusion] Ted-Jiang commented on issue #4459: Default parquet reader to reading 64K footer

2022-12-01 Thread GitBox
Ted-Jiang commented on issue #4459: URL: https://github.com/apache/arrow-datafusion/issues/4459#issuecomment-1334700643 Make sense to me, I think we have to notice the best practice to user keep footer size less than 64k. And i can not find a tool to read parquet footer size 😂 -- Th

[GitHub] [arrow] github-actions[bot] commented on pull request #14817: ARROW-18419: [C++] Update vendored fast_float

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14817: URL: https://github.com/apache/arrow/pull/14817#issuecomment-1334699035 https://issues.apache.org/jira/browse/ARROW-18419 -- 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] github-actions[bot] commented on pull request #14817: ARROW-18419: [C++] Update vendored fast_float

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14817: URL: https://github.com/apache/arrow/pull/14817#issuecomment-1334699049 :warning: Ticket **has not been started in JIRA**, please click 'Start Progress'. -- This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [arrow] kou opened a new pull request, #14817: ARROW-18419: [C++] Update vendored fast_float

2022-12-01 Thread GitBox
kou opened a new pull request, #14817: URL: https://github.com/apache/arrow/pull/14817 It's for https://github.com/fastfloat/fast_float/pull/147 . -- 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] yli1994 commented on issue #14726: pq.read_table("parquet files path", memory_map=True) still consume large memory space(200G file cost 200G memory and slow)

2022-12-01 Thread GitBox
yli1994 commented on issue #14726: URL: https://github.com/apache/arrow/issues/14726#issuecomment-1334690052 > If you want to reduce memory usage when reading a file, you should not read it as an entire table, but as a sequence of batches. See here: https://arrow.apache.org/docs/python/parq

[GitHub] [arrow] yli1994 commented on issue #14726: pq.read_table("parquet files path", memory_map=True) still consume large memory space(200G file cost 200G memory and slow)

2022-12-01 Thread GitBox
yli1994 commented on issue #14726: URL: https://github.com/apache/arrow/issues/14726#issuecomment-1334688842 > If you want to reduce memory usage when reading a file, you should not read it as an entire table, but as a sequence of batches. See here: https://arrow.apache.org/docs/python/parq

[GitHub] [arrow] wgtmac commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
wgtmac commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037733470 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,143 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. See the

[GitHub] [arrow] wgtmac commented on a diff in pull request #14803: ARROW-18420: [C++][Parquet] Introduce ColumnIndex & OffsetIndex

2022-12-01 Thread GitBox
wgtmac commented on code in PR #14803: URL: https://github.com/apache/arrow/pull/14803#discussion_r1037728861 ## cpp/src/parquet/page_index.h: ## @@ -0,0 +1,143 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements. See the

[GitHub] [arrow] ycyang-26 commented on issue #14798: [parquet go] write parquet data code sample

2022-12-01 Thread GitBox
ycyang-26 commented on issue #14798: URL: https://github.com/apache/arrow/issues/14798#issuecomment-1334680024 > There was discussion on the Zulip by Ashish Paliwal (i don't know his GH username) for adding Go examples to the Arrow Cookbooks but I don't believe anything came of it yet. It's

[GitHub] [arrow-adbc] lidavidm commented on pull request #208: ci: create GitHub release for RC tags

2022-12-01 Thread GitBox
lidavidm commented on PR #208: URL: https://github.com/apache/arrow-adbc/pull/208#issuecomment-1334675860 Aaaand now I need to list out the artifacts properly. https://github.com/lidavidm/arrow-adbc/actions/runs/3598246947 -- This is an automated message from the Apache Git Service. To re

[GitHub] [arrow] github-actions[bot] commented on pull request #14235: ARROW-17692: [R] Add support for building with system AWS SDK C++

2022-12-01 Thread GitBox
github-actions[bot] commented on PR #14235: URL: https://github.com/apache/arrow/pull/14235#issuecomment-1334667248 Revision: a608425a40d0e5e5fad3797ddbff11c1442541a7 Submitted crossbow builds: [ursacomputing/crossbow @ actions-2080a1b82e](https://github.com/ursacomputing/crossbow/bra

[GitHub] [arrow] kou commented on pull request #14235: ARROW-17692: [R] Add support for building with system AWS SDK C++

2022-12-01 Thread GitBox
kou commented on PR #14235: URL: https://github.com/apache/arrow/pull/14235#issuecomment-1334665340 @github-actions crossbow submit -g r -- 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 specif

[GitHub] [arrow-datafusion] HaoYang670 commented on issue #4378: The CLI panics when passing an invalid `explain` query

2022-12-01 Thread GitBox
HaoYang670 commented on issue #4378: URL: https://github.com/apache/arrow-datafusion/issues/4378#issuecomment-1334665315 I file https://github.com/sqlparser-rs/sqlparser-rs/issues/744 in the sqlparser-rs to fix this parsing bug -- This is an automated message from the Apache Git Service.

[GitHub] [arrow] ursabot commented on pull request #14793: GH-14780: [Go] Fix issues with IPC writing of sliced map/list arrays

2022-12-01 Thread GitBox
ursabot commented on PR #14793: URL: https://github.com/apache/arrow/pull/14793#issuecomment-1334663838 Benchmark runs are scheduled for baseline = 1c8853bfbb8d02abc2aafa1d19724f41090739cd and contender = 02bbcc5057302696227dc4a7be004c043ad7a68e. 02bbcc5057302696227dc4a7be004c043ad7a68e is

[GitHub] [arrow-datafusion] HaoYang670 closed issue #4452: Multiple EXPLAINs cause panic

2022-12-01 Thread GitBox
HaoYang670 closed issue #4452: Multiple EXPLAINs cause panic URL: https://github.com/apache/arrow-datafusion/issues/4452 -- 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 unsu

[GitHub] [arrow-datafusion] HaoYang670 commented on issue #4452: Multiple EXPLAINs cause panic

2022-12-01 Thread GitBox
HaoYang670 commented on issue #4452: URL: https://github.com/apache/arrow-datafusion/issues/4452#issuecomment-1334659379 I will close this ticket as #4429 has been merged -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [arrow-adbc] lidavidm commented on pull request #208: ci: create GitHub release for RC tags

2022-12-01 Thread GitBox
lidavidm commented on PR #208: URL: https://github.com/apache/arrow-adbc/pull/208#issuecomment-1334628216 Ah, I forgot - it needs some way to figure out the repository name. Trying again (specifying that explicitly): https://github.com/lidavidm/arrow-adbc/actions/runs/3597850328 -- This

[GitHub] [arrow-adbc] kou commented on pull request #208: ci: create GitHub release for RC tags

2022-12-01 Thread GitBox
kou commented on PR #208: URL: https://github.com/apache/arrow-adbc/pull/208#issuecomment-1334610088 It seems that we need `actions/checkout` to use `gh`: https://github.com/lidavidm/arrow-adbc/actions/runs/3596960711/jobs/6058660390 ```text failed to run git: fatal: not a g

[GitHub] [arrow] ursabot commented on pull request #14777: ARROW-18112: [Go] Remaining Scalar Arithmetic

2022-12-01 Thread GitBox
ursabot commented on PR #14777: URL: https://github.com/apache/arrow/pull/14777#issuecomment-1334575150 Benchmark runs are scheduled for baseline = 1d9f7781acf5e5b3679ac2d2aee9e837435ac87e and contender = 1c8853bfbb8d02abc2aafa1d19724f41090739cd. 1c8853bfbb8d02abc2aafa1d19724f41090739cd is

[GitHub] [arrow-datafusion-python] dependabot[bot] opened a new pull request, #101: build(deps): bump pyo3 from 0.17.2 to 0.17.3

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #101: URL: https://github.com/apache/arrow-datafusion-python/pull/101 Bumps [pyo3](https://github.com/pyo3/pyo3) from 0.17.2 to 0.17.3. Release notes Sourced from https://github.com/pyo3/pyo3/releases";>pyo3's releases. PyO3 0.17.3

[GitHub] [arrow-datafusion-python] dependabot[bot] opened a new pull request, #99: build(deps): bump tokio from 1.21.2 to 1.22.0

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #99: URL: https://github.com/apache/arrow-datafusion-python/pull/99 Bumps [tokio](https://github.com/tokio-rs/tokio) from 1.21.2 to 1.22.0. Release notes Sourced from https://github.com/tokio-rs/tokio/releases";>tokio's releases. Tok

[GitHub] [arrow-datafusion-python] dependabot[bot] opened a new pull request, #100: build(deps): bump async-trait from 0.1.57 to 0.1.59

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #100: URL: https://github.com/apache/arrow-datafusion-python/pull/100 Bumps [async-trait](https://github.com/dtolnay/async-trait) from 0.1.57 to 0.1.59. Release notes Sourced from https://github.com/dtolnay/async-trait/releases";>async-tra

[GitHub] [arrow-datafusion-python] dependabot[bot] opened a new pull request, #102: build(deps): bump futures from 0.3.24 to 0.3.25

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #102: URL: https://github.com/apache/arrow-datafusion-python/pull/102 Bumps [futures](https://github.com/rust-lang/futures-rs) from 0.3.24 to 0.3.25. Release notes Sourced from https://github.com/rust-lang/futures-rs/releases";>futures's

[GitHub] [arrow-datafusion-python] dependabot[bot] opened a new pull request, #96: build(deps): bump uuid from 0.8.2 to 1.2.1

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #96: URL: https://github.com/apache/arrow-datafusion-python/pull/96 Bumps [uuid](https://github.com/uuid-rs/uuid) from 0.8.2 to 1.2.1. Release notes Sourced from https://github.com/uuid-rs/uuid/releases";>uuid's releases. 1.2.1 Wh

[GitHub] [arrow-datafusion-python] dependabot[bot] opened a new pull request, #97: build(deps): bump rand from 0.7.3 to 0.8.5

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #97: URL: https://github.com/apache/arrow-datafusion-python/pull/97 Bumps [rand](https://github.com/rust-random/rand) from 0.7.3 to 0.8.5. Changelog Sourced from https://github.com/rust-random/rand/blob/master/CHANGELOG.md";>rand's changel

[GitHub] [arrow-datafusion-python] dependabot[bot] opened a new pull request, #98: build(deps): bump mimalloc from 0.1.30 to 0.1.32

2022-12-01 Thread GitBox
dependabot[bot] opened a new pull request, #98: URL: https://github.com/apache/arrow-datafusion-python/pull/98 Bumps [mimalloc](https://github.com/purpleprotocol/mimalloc_rust) from 0.1.30 to 0.1.32. Release notes Sourced from https://github.com/purpleprotocol/mimalloc_rust/release

[GitHub] [arrow-datafusion-python] andygrove merged pull request #93: Fixes #81 - Add CI workflow for source distribution

2022-12-01 Thread GitBox
andygrove merged PR #93: URL: https://github.com/apache/arrow-datafusion-python/pull/93 -- 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-unsubscr.

[GitHub] [arrow-datafusion-python] andygrove closed issue #81: Build Python source distribution in GitHub workflow

2022-12-01 Thread GitBox
andygrove closed issue #81: Build Python source distribution in GitHub workflow URL: https://github.com/apache/arrow-datafusion-python/issues/81 -- 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 sp

[GitHub] [arrow-datafusion-python] andygrove closed pull request #92: configure publishing of asf-site branch

2022-12-01 Thread GitBox
andygrove closed pull request #92: configure publishing of asf-site branch URL: https://github.com/apache/arrow-datafusion-python/pull/92 -- 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

[GitHub] [arrow-datafusion-python] andygrove closed issue #94: Github actions produce a lot of warnings

2022-12-01 Thread GitBox
andygrove closed issue #94: Github actions produce a lot of warnings URL: https://github.com/apache/arrow-datafusion-python/issues/94 -- 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 comm

[GitHub] [arrow-datafusion-python] andygrove merged pull request #95: Fix GitHub actions warnings

2022-12-01 Thread GitBox
andygrove merged PR #95: URL: https://github.com/apache/arrow-datafusion-python/pull/95 -- 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-unsubscr.

[GitHub] [arrow-datafusion-python] andygrove commented on a diff in pull request #93: Fixes #81 - Add CI workflow for source distribution

2022-12-01 Thread GitBox
andygrove commented on code in PR #93: URL: https://github.com/apache/arrow-datafusion-python/pull/93#discussion_r1037640596 ## pyproject.toml: ## @@ -55,4 +55,9 @@ repository = "https://github.com/apache/arrow-datafusion-python"; profile = "black" [tool.maturin] -sdist-in

[GitHub] [arrow-datafusion] alamb commented on a diff in pull request #4472: Drive sqllogictest runner on directory contents rather than hard coded list

2022-12-01 Thread GitBox
alamb commented on code in PR #4472: URL: https://github.com/apache/arrow-datafusion/pull/4472#discussion_r1037612827 ## datafusion/core/tests/sqllogictests/src/main.rs: ## @@ -29,53 +29,18 @@ mod setup; mod utils; const TEST_DIRECTORY: &str = "tests/sqllogictests/test_files

[GitHub] [arrow-datafusion] alamb opened a new pull request, #4472: Drive sqllogictest runner on directory contents rather than hard coded list

2022-12-01 Thread GitBox
alamb opened a new pull request, #4472: URL: https://github.com/apache/arrow-datafusion/pull/4472 # Which issue does this PR close? Closes https://github.com/apache/arrow-datafusion/issues/4471 # Rationale for this change Basically I want to be able to add new files easil

[GitHub] [arrow-datafusion] alamb opened a new issue, #4471: Drive sqllogictest runner on directory contents rather than hard coded list

2022-12-01 Thread GitBox
alamb opened a new issue, #4471: URL: https://github.com/apache/arrow-datafusion/issues/4471 **Is your feature request related to a problem or challenge? Please describe what you are trying to do.** I want to add new tests based on the sqllogictest runner However, doing so now requ

[GitHub] [arrow-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037606650 ## dev/release/04-binary-download.sh: ## @@ -0,0 +1,59 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor li

[GitHub] [arrow-adbc] lidavidm commented on pull request #208: ci: create GitHub release for RC tags

2022-12-01 Thread GitBox
lidavidm commented on PR #208: URL: https://github.com/apache/arrow-adbc/pull/208#issuecomment-1334512182 Test run here: https://github.com/lidavidm/arrow-adbc/actions/runs/3596960711 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [arrow-rs] wjones127 commented on a diff in pull request #3236: fix(object_store,gcp): test copy_if_not_exist

2022-12-01 Thread GitBox
wjones127 commented on code in PR #3236: URL: https://github.com/apache/arrow-rs/pull/3236#discussion_r1037600571 ## object_store/src/gcp/mod.rs: ## @@ -420,10 +430,25 @@ impl GoogleCloudStorageClient { builder .bearer_auth(token) +.header(hea

[GitHub] [arrow-datafusion] andygrove opened a new pull request, #4470: Prepare for 15.0.0 Release

2022-12-01 Thread GitBox
andygrove opened a new pull request, #4470: URL: https://github.com/apache/arrow-datafusion/pull/4470 # Which issue does this PR close? Closes #. # Rationale for this change # What changes are included in this PR? # Are these changes tested?

[GitHub] [arrow-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037593049 ## dev/release/04-binary-download.sh: ## @@ -0,0 +1,59 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor li

[GitHub] [arrow-adbc] kou commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
kou commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037584656 ## dev/release/04-binary-download.sh: ## @@ -0,0 +1,59 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

[GitHub] [arrow-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037579484 ## dev/release/04-binary-download.sh: ## @@ -0,0 +1,59 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor li

[GitHub] [arrow-datafusion] alamb commented on pull request #4469: MINOR: Add note about sqllogictest to contributor guide

2022-12-01 Thread GitBox
alamb commented on PR #4469: URL: https://github.com/apache/arrow-datafusion/pull/4469#issuecomment-1334467816 cc @xudong963 -- 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

[GitHub] [arrow-datafusion] alamb opened a new pull request, #4469: MINOR: Add note about sqllogictest to contributor guide

2022-12-01 Thread GitBox
alamb opened a new pull request, #4469: URL: https://github.com/apache/arrow-datafusion/pull/4469 # Which issue does this PR close? re https://github.com/apache/arrow-datafusion/issues/4460 # Rationale for this change make the harness added in https://github.com/apache/a

[GitHub] [arrow-adbc] kou commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
kou commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037572454 ## dev/release/06-java-upload.sh: ## @@ -0,0 +1,133 @@ +#!/usr/bin/env bash Review Comment: OK. I work on it: https://github.com/apache/arrow/issues/14816 -- This

[GitHub] [arrow-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037569568 ## dev/release/04-binary-download.sh: ## @@ -0,0 +1,59 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor li

[GitHub] [arrow-adbc] kou commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
kou commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037569812 ## docs/source/development/releasing.rst: ## @@ -0,0 +1,255 @@ +.. Licensed to the Apache Software Foundation (ASF) under one +.. or more contributor license agreements.

[GitHub] [arrow-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037568982 ## dev/release/03-binary-submit.sh: ## @@ -0,0 +1,64 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor lice

[GitHub] [arrow-datafusion] alamb commented on issue #3105: Median aggregation using DataFrame panics

2022-12-01 Thread GitBox
alamb commented on issue #3105: URL: https://github.com/apache/arrow-datafusion/issues/3105#issuecomment-1334455846 We are hitting this with IOx so I plan to fix it -- 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-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037564027 ## docs/source/development/releasing.rst: ## @@ -0,0 +1,255 @@ +.. Licensed to the Apache Software Foundation (ASF) under one +.. or more contributor license agreemen

[GitHub] [arrow-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037563502 ## dev/release/06-java-upload.sh: ## @@ -0,0 +1,133 @@ +#!/usr/bin/env bash Review Comment: Sounds good to me. That would be much appreciated. -- This is an a

[GitHub] [arrow-adbc] lidavidm commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
lidavidm commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037562792 ## dev/release/01-prepare.sh: ## @@ -0,0 +1,81 @@ +#!/usr/bin/env bash +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

[GitHub] [arrow-adbc] kou commented on a diff in pull request #174: chore: set up release process

2022-12-01 Thread GitBox
kou commented on code in PR #174: URL: https://github.com/apache/arrow-adbc/pull/174#discussion_r1037545137 ## dev/release/02-source.sh: ## @@ -0,0 +1,110 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreeme

[GitHub] [arrow-rs] alamb commented on a diff in pull request #3253: Update object_store version to 0.5.2 and add CHANGELOG

2022-12-01 Thread GitBox
alamb commented on code in PR #3253: URL: https://github.com/apache/arrow-rs/pull/3253#discussion_r1037553149 ## object_store/dev/release/update_change_log.sh: ## @@ -49,8 +49,8 @@ docker run -it --rm -e CHANGELOG_GITHUB_TOKEN="$CHANGELOG_GITHUB_TOKEN" -v "$(pw --cache-fil

[GitHub] [arrow] ursabot commented on pull request #14794: GH-14787: [Java] [Doc] Update table.rst

2022-12-01 Thread GitBox
ursabot commented on PR #14794: URL: https://github.com/apache/arrow/pull/14794#issuecomment-1334424325 Benchmark runs are scheduled for baseline = 92050336cbe6a36c5b9b3f695cf58cc595661e51 and contender = 1d9f7781acf5e5b3679ac2d2aee9e837435ac87e. 1d9f7781acf5e5b3679ac2d2aee9e837435ac87e is

[GitHub] [arrow-rs] alamb commented on pull request #2891: Add experimental AWS_PROFILE support (#2178)

2022-12-01 Thread GitBox
alamb commented on PR #2891: URL: https://github.com/apache/arrow-rs/pull/2891#issuecomment-1334417456 https://github.com/apache/arrow-rs/issues/3229 -- 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

  1   2   3   4   >