arw2019 opened a new pull request #8816:
URL: https://github.com/apache/arrow/pull/8816
Only relocation - none of the tests are touched.
cc @jorisvandenbossche
This is an automated message from the Apache Git Service.
andygrove commented on a change in pull request #8815:
URL: https://github.com/apache/arrow/pull/8815#discussion_r533821974
##
File path: rust/datafusion/src/lib.rs
##
@@ -52,6 +52,7 @@
clippy::useless_format,
clippy::zero_prefixed_literal
)]
kou commented on pull request #8817:
URL: https://github.com/apache/arrow/pull/8817#issuecomment-736959349
@github-actions crossbow submit centos-*
This is an automated message from the Apache Git Service.
To respond to the
kou opened a new pull request #8817:
URL: https://github.com/apache/arrow/pull/8817
Because CentOS 6 reached EOL at 2020-11-30.
This is an automated message from the Apache Git Service.
To respond to the message, please log
github-actions[bot] commented on pull request #8817:
URL: https://github.com/apache/arrow/pull/8817#issuecomment-736960561
Revision: b1c4fdba3923bb1ed2ff98c52d6129fe4dccbdad
Submitted crossbow builds: [ursa-labs/crossbow @
andygrove commented on a change in pull request #8794:
URL: https://github.com/apache/arrow/pull/8794#discussion_r533816728
##
File path: rust/arrow/src/compute/kernels/cast.rs
##
@@ -376,6 +378,27 @@ pub fn cast(array: , to_type: ) ->
Result {
Int64 =>
andygrove commented on a change in pull request #8815:
URL: https://github.com/apache/arrow/pull/8815#discussion_r533821974
##
File path: rust/datafusion/src/lib.rs
##
@@ -52,6 +52,7 @@
clippy::useless_format,
clippy::zero_prefixed_literal
)]
nealrichardson commented on a change in pull request #8757:
URL: https://github.com/apache/arrow/pull/8757#discussion_r533838250
##
File path: cpp/thirdparty/versions.txt
##
@@ -32,14 +32,17 @@ ARROW_BOOST_BUILD_VERSION=1.71.0
ARROW_BROTLI_BUILD_VERSION=v1.0.7
github-actions[bot] commented on pull request #8817:
URL: https://github.com/apache/arrow/pull/8817#issuecomment-736965696
https://issues.apache.org/jira/browse/ARROW-10786
This is an automated message from the Apache Git
andygrove commented on a change in pull request #8815:
URL: https://github.com/apache/arrow/pull/8815#discussion_r533821182
##
File path: rust/datafusion/tests/sql.rs
##
@@ -1496,6 +1496,25 @@ async fn csv_query_sum_cast() {
execute( ctx, sql).await;
}
+#[tokio::test]
nealrichardson closed pull request #8813:
URL: https://github.com/apache/arrow/pull/8813
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
github-actions[bot] commented on pull request #8815:
URL: https://github.com/apache/arrow/pull/8815#issuecomment-736921960
https://issues.apache.org/jira/browse/ARROW-10753
This is an automated message from the Apache Git
github-actions[bot] commented on pull request #8816:
URL: https://github.com/apache/arrow/pull/8816#issuecomment-736921959
https://issues.apache.org/jira/browse/ARROW-9027
This is an automated message from the Apache Git
GregBowyer commented on pull request #8698:
URL: https://github.com/apache/arrow/pull/8698#issuecomment-736938052
I have been working on this w.r.t performance, I think I have most parts
performing better than the original. I am running off clean benchmarks right
now to validate.
nealrichardson commented on a change in pull request #8813:
URL: https://github.com/apache/arrow/pull/8813#discussion_r533785013
##
File path: r/vignettes/install.Rmd
##
@@ -284,8 +284,15 @@ so that we can attempt to improve the script.
## Known installation issues
* On
andygrove commented on a change in pull request #8815:
URL: https://github.com/apache/arrow/pull/8815#discussion_r533820801
##
File path: rust/datafusion/src/sql/planner.rs
##
@@ -638,10 +638,13 @@ impl<'a, S: SchemaProvider> SqlToRel<'a, S> {
nevi-me commented on pull request #8800:
URL: https://github.com/apache/arrow/pull/8800#issuecomment-737022972
> Looks like an unrelated failure in the CI
The failure's indeed unrelated, it's something we subsequently fixed, but is
still failing because it's not yet rebased in this
nealrichardson commented on a change in pull request #8813:
URL: https://github.com/apache/arrow/pull/8813#discussion_r533676787
##
File path: r/vignettes/install.Rmd
##
@@ -284,7 +284,11 @@ so that we can attempt to improve the script.
## Known installation issues
* On
yordan-pavlov commented on a change in pull request #8794:
URL: https://github.com/apache/arrow/pull/8794#discussion_r533682639
##
File path: rust/arrow/src/compute/kernels/cast.rs
##
@@ -376,6 +378,27 @@ pub fn cast(array: , to_type: ) ->
Result {
Int64 =>
mattpollock commented on a change in pull request #8813:
URL: https://github.com/apache/arrow/pull/8813#discussion_r533674921
##
File path: r/vignettes/install.Rmd
##
@@ -284,7 +284,11 @@ so that we can attempt to improve the script.
## Known installation issues
* On
kou commented on a change in pull request #8813:
URL: https://github.com/apache/arrow/pull/8813#discussion_r533697120
##
File path: r/vignettes/install.Rmd
##
@@ -284,7 +284,11 @@ so that we can attempt to improve the script.
## Known installation issues
* On CentOS, if
yordan-pavlov commented on a change in pull request #8794:
URL: https://github.com/apache/arrow/pull/8794#discussion_r533701070
##
File path: rust/arrow/src/compute/kernels/cast.rs
##
@@ -2720,6 +2743,42 @@ mod tests {
.collect()
}
+#[test]
+fn
nealrichardson commented on a change in pull request #8808:
URL: https://github.com/apache/arrow/pull/8808#discussion_r533643340
##
File path: r/DESCRIPTION
##
@@ -48,7 +48,7 @@ Suggests:
rmarkdown,
testthat,
tibble
-LinkingTo: cpp11
+LinkingTo: cpp11 (>= 0.2.0)
nealrichardson closed pull request #8808:
URL: https://github.com/apache/arrow/pull/8808
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
mattpollock opened a new pull request #8813:
URL: https://github.com/apache/arrow/pull/8813
Update known installation issues vignette per discussion in ARROW-10780
This is an automated message from the Apache Git Service.
To
bkietz commented on a change in pull request #8770:
URL: https://github.com/apache/arrow/pull/8770#discussion_r533640298
##
File path: cpp/src/arrow/util/bit_util_test.cc
##
@@ -66,6 +66,15 @@ using internal::InvertBitmap;
using ::testing::ElementsAreArray;
+namespace
kou commented on pull request #8782:
URL: https://github.com/apache/arrow/pull/8782#issuecomment-736770394
I think so.
We need a fix like
https://github.com/apache/arrow/pull/8782#issuecomment-736217882 for
`cpp/src/parquet/column_writer_test.cc`.
bkietz commented on a change in pull request #8777:
URL: https://github.com/apache/arrow/pull/8777#discussion_r533624625
##
File path: cpp/src/arrow/compute/kernels/vector_selection.cc
##
@@ -1838,19 +1838,113 @@ Result> FilterTable(const
Table& table, const Datum& filt
if
bkietz commented on a change in pull request #8761:
URL: https://github.com/apache/arrow/pull/8761#discussion_r533626348
##
File path: cpp/src/arrow/compute/kernels/vector_sort.cc
##
@@ -238,21 +239,9 @@ inline void VisitRawValuesInline(const ArrayType& values,
bkietz closed pull request #8613:
URL: https://github.com/apache/arrow/pull/8613
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
nealrichardson commented on a change in pull request #8757:
URL: https://github.com/apache/arrow/pull/8757#discussion_r533651013
##
File path: cpp/thirdparty/versions.txt
##
@@ -32,14 +32,17 @@ ARROW_BOOST_BUILD_VERSION=1.71.0
ARROW_BROTLI_BUILD_VERSION=v1.0.7
nealrichardson commented on a change in pull request #8813:
URL: https://github.com/apache/arrow/pull/8813#discussion_r533657147
##
File path: r/vignettes/install.Rmd
##
@@ -284,7 +284,11 @@ so that we can attempt to improve the script.
## Known installation issues
* On
bluss edited a comment on pull request #7734:
URL: https://github.com/apache/arrow/pull/7734#issuecomment-736685727
It is claimed https://issues.apache.org/jira/browse/ARROW-8480 is solved by
this PR, but this PR does not add checks for allocation faliure, and there are
still bugs to spot
jorgecarleitao commented on a change in pull request #8401:
URL: https://github.com/apache/arrow/pull/8401#discussion_r533614032
##
File path: rust/arrow-pyarrow-integration-testing/tests/test_sql.py
##
@@ -0,0 +1,61 @@
+# -*- coding: utf-8 -*-
+# Licensed to the Apache
andygrove closed pull request #8807:
URL: https://github.com/apache/arrow/pull/8807
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
nevi-me closed pull request #8804:
URL: https://github.com/apache/arrow/pull/8804
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
vikashsingh009 opened a new issue #8812:
URL: https://github.com/apache/arrow/issues/8812
I have imported arrowjs in typescript like
import { Table } from 'apache-arrow';
const arrow = readFileSync('simple.arrow');
const table = Table.from([arrow]);
but when i
jorgecarleitao commented on a change in pull request #8401:
URL: https://github.com/apache/arrow/pull/8401#discussion_r533521705
##
File path: rust/arrow-c-integration/src/lib.rs
##
@@ -0,0 +1,162 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more
Dandandan commented on pull request #8800:
URL: https://github.com/apache/arrow/pull/8800#issuecomment-736683582
Looks like an unrelated failure in the CI
This is an automated message from the Apache Git Service.
To respond
bluss edited a comment on pull request #7734:
URL: https://github.com/apache/arrow/pull/7734#issuecomment-736685727
It is claimed https://issues.apache.org/jira/browse/ARROW-8480 is solved by
this PR, but this PR does not add checks for allocation faliure, and there are
still bugs due to
bluss commented on pull request #7734:
URL: https://github.com/apache/arrow/pull/7734#issuecomment-736685727
It is claimed https://issues.apache.org/jira/browse/ARROW-8480 is solved by
this PR, but this PR does not add checks for allocation faliure, and there are
still bugs due to that,
github-actions[bot] commented on pull request #8813:
URL: https://github.com/apache/arrow/pull/8813#issuecomment-736820773
https://issues.apache.org/jira/browse/ARROW-10780
This is an automated message from the Apache Git
jorisvandenbossche commented on pull request #8704:
URL: https://github.com/apache/arrow/pull/8704#issuecomment-736823311
I added a test, this should be good now
This is an automated message from the Apache Git Service.
To
Dandandan opened a new pull request #8814:
URL: https://github.com/apache/arrow/pull/8814
Further optimizes take string to benefit from creating a buffer directly.
Interestingly, it seems it is still faster to copy a `Vec` than to use a
buffer for the the values. Presumably
josiahyan commented on a change in pull request #8757:
URL: https://github.com/apache/arrow/pull/8757#discussion_r533728016
##
File path: cpp/thirdparty/versions.txt
##
@@ -32,14 +32,17 @@ ARROW_BOOST_BUILD_VERSION=1.71.0
ARROW_BROTLI_BUILD_VERSION=v1.0.7
josiahyan commented on a change in pull request #8757:
URL: https://github.com/apache/arrow/pull/8757#discussion_r533728016
##
File path: cpp/thirdparty/versions.txt
##
@@ -32,14 +32,17 @@ ARROW_BOOST_BUILD_VERSION=1.71.0
ARROW_BROTLI_BUILD_VERSION=v1.0.7
github-actions[bot] commented on pull request #8814:
URL: https://github.com/apache/arrow/pull/8814#issuecomment-736850611
https://issues.apache.org/jira/browse/ARROW-10785
This is an automated message from the Apache Git
velvia commented on pull request #8815:
URL: https://github.com/apache/arrow/pull/8815#issuecomment-736864975
/cc @jorgecarleitao who last changed this part of planner.rs
This is an automated message from the Apache Git
velvia opened a new pull request #8815:
URL: https://github.com/apache/arrow/pull/8815
Currently, DataFusion SQL statements that compare negative numbers result in
an exception, as negative numbers are incorrectly parsed. The error thrown is
`InternalError("SQL binary operator cannot be
github-actions[bot] commented on pull request #8809:
URL: https://github.com/apache/arrow/pull/8809#issuecomment-736372323
https://issues.apache.org/jira/browse/ARROW-10778
This is an automated message from the Apache Git
maartenbreddels commented on pull request #8755:
URL: https://github.com/apache/arrow/pull/8755#issuecomment-736369145
I think the failure is unrelated.
I'm really excited about this (seemingly trivial) feature, this gives such a
massive performance improvement (equal to memory
pitrou commented on a change in pull request #8755:
URL: https://github.com/apache/arrow/pull/8755#discussion_r533193607
##
File path: cpp/src/arrow/python/io.cc
##
@@ -199,25 +219,32 @@ Result PyReadableFile::Read(int64_t nbytes,
void* out) {
PyObject* bytes_obj =
maartenbreddels commented on a change in pull request #8755:
URL: https://github.com/apache/arrow/pull/8755#discussion_r533159508
##
File path: python/pyarrow/tests/test_io.py
##
@@ -163,6 +163,34 @@ def test_python_file_readinto():
assert len(dst_buf) == length
pitrou commented on a change in pull request #8757:
URL: https://github.com/apache/arrow/pull/8757#discussion_r533241989
##
File path: ci/docker/conda-cpp.dockerfile
##
@@ -73,6 +73,7 @@ ENV ARROW_BUILD_TESTS=ON \
ARROW_PARQUET=ON \
ARROW_PLASMA=ON \
ARROW_S3=ON
pitrou commented on pull request #8755:
URL: https://github.com/apache/arrow/pull/8755#issuecomment-736373876
> I think the failure is unrelated.
It is. We're dropping Python 3.5 support soon.
This is an automated
pitrou closed pull request #8761:
URL: https://github.com/apache/arrow/pull/8761
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
pitrou commented on pull request #8704:
URL: https://github.com/apache/arrow/pull/8704#issuecomment-737046252
Thank you @jorisvandenbossche !
This is an automated message from the Apache Git Service.
To respond to the
pitrou commented on a change in pull request #8761:
URL: https://github.com/apache/arrow/pull/8761#discussion_r533951860
##
File path: cpp/src/arrow/compute/kernels/vector_sort.cc
##
@@ -238,21 +239,9 @@ inline void VisitRawValuesInline(const ArrayType& values,
pitrou commented on a change in pull request #8761:
URL: https://github.com/apache/arrow/pull/8761#discussion_r533951860
##
File path: cpp/src/arrow/compute/kernels/vector_sort.cc
##
@@ -238,21 +239,9 @@ inline void VisitRawValuesInline(const ArrayType& values,
pitrou commented on a change in pull request #8770:
URL: https://github.com/apache/arrow/pull/8770#discussion_r533955521
##
File path: cpp/src/arrow/util/bit_run_reader.h
##
@@ -166,7 +167,350 @@ class ARROW_EXPORT BitRunReader {
using BitRunReader = BitRunReaderLinear;
pitrou closed pull request #8704:
URL: https://github.com/apache/arrow/pull/8704
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
pitrou closed pull request #8755:
URL: https://github.com/apache/arrow/pull/8755
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
Dandandan commented on pull request #8743:
URL: https://github.com/apache/arrow/pull/8743#issuecomment-736452020
Awesome that the "main" join support is now in!
This is an automated message from the Apache Git Service.
To
github-actions[bot] commented on pull request #8810:
URL: https://github.com/apache/arrow/pull/8810#issuecomment-736534626
https://issues.apache.org/jira/browse/ARROW-10779
This is an automated message from the Apache Git
pitrou commented on pull request #8776:
URL: https://github.com/apache/arrow/pull/8776#issuecomment-736392159
@emkornfield opined on this on the ML.
This is an automated message from the Apache Git Service.
To respond to
projjal closed pull request #8810:
URL: https://github.com/apache/arrow/pull/8810
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
jorisvandenbossche closed pull request #8809:
URL: https://github.com/apache/arrow/pull/8809
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
projjal opened a new pull request #8810:
URL: https://github.com/apache/arrow/pull/8810
UnionListWriter#writeNull currently increments the index in the inner writer
and doesn't unset the validity at the particular index. So if the validity at
that index was already set (like due to
jorisvandenbossche commented on a change in pull request #8809:
URL: https://github.com/apache/arrow/pull/8809#discussion_r533291547
##
File path: python/pyarrow/_dataset.pyx
##
@@ -929,7 +929,8 @@ class RowGroupInfo:
def name_stats(i):
col =
josiahyan commented on a change in pull request #8757:
URL: https://github.com/apache/arrow/pull/8757#discussion_r533463385
##
File path: ci/docker/conda-cpp.dockerfile
##
@@ -73,6 +73,7 @@ ENV ARROW_BUILD_TESTS=ON \
ARROW_PARQUET=ON \
ARROW_PLASMA=ON \
pitrou opened a new pull request #8811:
URL: https://github.com/apache/arrow/pull/8811
* Add an ipc::IpcWriteOptions member to govern emission of dictionary deltas.
If the option is enabled, deltas are detected by checking whether the new
dictionary starts with the last
pitrou commented on pull request #8776:
URL: https://github.com/apache/arrow/pull/8776#issuecomment-736622731
@xhochy, @kszucs, could one of you rewrite this?
This is an automated message from the Apache Git Service.
To
nevi-me commented on pull request #8800:
URL: https://github.com/apache/arrow/pull/8800#issuecomment-736605119
I've rerun the integration CI job, we can merge this after we get
confirmation
This is an automated message from
github-actions[bot] commented on pull request #8811:
URL: https://github.com/apache/arrow/pull/8811#issuecomment-736628856
https://issues.apache.org/jira/browse/ARROW-6883
This is an automated message from the Apache Git
josiahyan commented on pull request #8757:
URL: https://github.com/apache/arrow/pull/8757#issuecomment-736638562
@github-actions rebase
This is an automated message from the Apache Git Service.
To respond to the message,
andygrove closed pull request #8803:
URL: https://github.com/apache/arrow/pull/8803
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
nevi-me closed pull request #8799:
URL: https://github.com/apache/arrow/pull/8799
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
xhochy closed pull request #8778:
URL: https://github.com/apache/arrow/pull/8778
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
andygrove closed pull request #8798:
URL: https://github.com/apache/arrow/pull/8798
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
79 matches
Mail list logo