nealrichardson commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-889950166
I've reproduced the error on https://github.com/apache/arrow/pull/10839, so
I'll merge that (assuming I haven't broken other things) and then you can
rebase this, run the
bkietz commented on a change in pull request #10802:
URL: https://github.com/apache/arrow/pull/10802#discussion_r679994244
##
File path: cpp/src/arrow/compute/kernels/vector_selection.cc
##
@@ -2146,6 +2147,144 @@ class TakeMetaFunction : public MetaFunction {
}
};
+// --
bkietz commented on a change in pull request #10802:
URL: https://github.com/apache/arrow/pull/10802#discussion_r680002741
##
File path: cpp/src/arrow/compute/kernels/vector_selection.cc
##
@@ -2146,6 +2147,144 @@ class TakeMetaFunction : public MetaFunction {
}
};
+// --
nirandaperera commented on a change in pull request #10802:
URL: https://github.com/apache/arrow/pull/10802#discussion_r680002827
##
File path: cpp/src/arrow/compute/kernels/vector_selection.cc
##
@@ -2146,6 +2147,144 @@ class TakeMetaFunction : public MetaFunction {
}
};
thisisnic commented on a change in pull request #10710:
URL: https://github.com/apache/arrow/pull/10710#discussion_r680012443
##
File path: r/inst/build_arrow_static.sh
##
@@ -45,6 +45,19 @@ else
ARROW_DEFAULT_PARAM="OFF"
fi
+if [ "$R_STATIC_DEPENDENCY_SOURCE" = "AUTO" ];
dongjoon-hyun commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-889969658
Also cc @sunchao .
--
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 spec
nealrichardson closed pull request #10839:
URL: https://github.com/apache/arrow/pull/10839
--
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-
nealrichardson commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-889973030
ok @nirandaperera, merged, please rebase and then do `@github-actions
crossbow submit test-r-linux-as-cran`
--
This is an automated message from the Apache Git Service.
nirandaperera commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-889974932
@github-actions crossbow submit test-r-linux-as-cran
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
github-actions[bot] commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-889975532
Revision: fe0b1ec8c425c963a7470ace9fd3af14674b37dd
Submitted crossbow builds: [ursacomputing/crossbow @
actions-678](https://github.com/ursacomputing/crossbow/
emkornfield commented on pull request #10627:
URL: https://github.com/apache/arrow/pull/10627#issuecomment-889976082
sorry I have had less time then I would have liked recently for Arrow
reviews, will try to get to this soon.
--
This is an automated message from the Apache Git Service.
T
emkornfield commented on pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#issuecomment-889976220
sorry I have had less time then I would have liked recently for Arrow
reviews, will try to get to this soon.
--
This is an automated message from the Apache Git Service.
T
emkornfield commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680022978
##
File path: go/parquet/internal/encoding/encoding_benchmarks_test.go
##
@@ -0,0 +1,461 @@
+// Licensed to the Apache Software Foundation (ASF) under
emkornfield commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680026208
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
emkornfield commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680027195
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
emkornfield commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680027388
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
nealrichardson commented on a change in pull request #10710:
URL: https://github.com/apache/arrow/pull/10710#discussion_r680027557
##
File path: r/inst/build_arrow_static.sh
##
@@ -45,6 +45,19 @@ else
ARROW_DEFAULT_PARAM="OFF"
fi
+if [ "$R_STATIC_DEPENDENCY_SOURCE" = "AUT
zeroshade commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680028297
##
File path: go/parquet/internal/encoding/encoding_benchmarks_test.go
##
@@ -0,0 +1,461 @@
+// Licensed to the Apache Software Foundation (ASF) under o
emkornfield commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680028272
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
nealrichardson commented on a change in pull request #10710:
URL: https://github.com/apache/arrow/pull/10710#discussion_r680028253
##
File path: r/inst/build_arrow_static.sh
##
@@ -45,6 +45,19 @@ else
ARROW_DEFAULT_PARAM="OFF"
fi
+if [ "$R_STATIC_DEPENDENCY_SOURCE" = "AUT
zeroshade commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680029295
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more con
nealrichardson opened a new pull request #10841:
URL: https://github.com/apache/arrow/pull/10841
--
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: g
nealrichardson commented on a change in pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#discussion_r680032585
##
File path: ci/scripts/r_deps.sh
##
@@ -25,7 +25,12 @@ source_dir=${1}/r
pushd ${source_dir}
# Install R package dependencies
-${R_BIN} -e "i
github-actions[bot] commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-889985671
https://issues.apache.org/jira/browse/ARROW-13511
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
nealrichardson commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-889985861
@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 abov
github-actions[bot] commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-889986428
Revision: 8c6073efab365dde7b6a373c0a13019992a3023a
Submitted crossbow builds: [ursacomputing/crossbow @
actions-679](https://github.com/ursacomputing/crossbow/
emkornfield commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680034150
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
nealrichardson commented on pull request #10840:
URL: https://github.com/apache/arrow/pull/10840#issuecomment-889987032
As best I can tell, it's not that the job is timing out, it's that the job
is running without printing anything to stdout for too long (this is what
Travis CI would do).
zeroshade commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680037198
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more con
jhorstmann commented on pull request #793:
URL: https://github.com/apache/arrow-datafusion/pull/793#issuecomment-889992552
Looks good. I was trying to come up with an example where two distinct keys
would end up as the same encoding but could not find any because any variable
length types
diegodfrf opened a new pull request #10842:
URL: https://github.com/apache/arrow/pull/10842
Set initial values to DayMilliseconds with 0
--
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 speci
github-actions[bot] commented on pull request #10842:
URL: https://github.com/apache/arrow/pull/10842#issuecomment-89320
https://issues.apache.org/jira/browse/ARROW-13469
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
thisisnic commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-890009339
I think building the things in `Suggests` has led to the build rebuilding
the markdown content, which then causes it to fail due to other dependencies in
the vignettes.
--
zeroshade commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680074956
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more con
zeroshade commented on a change in pull request #10716:
URL: https://github.com/apache/arrow/pull/10716#discussion_r680075490
##
File path: go/parquet/internal/encoding/levels.go
##
@@ -0,0 +1,284 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more con
thisisnic edited a comment on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-890009339
I think building the things in `Suggests` has led to the build rebuilding
the markdown content, which then causes `test-r-without-arrow` to fail due to
other dependenci
nealrichardson commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-890013534
> I think building the things in `Suggests` has led to the build rebuilding
the markdown content, which then causes `test-r-without-arrow` to fail due to
other dependenci
thisisnic commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-890014347
> > I think building the things in `Suggests` has led to the build
rebuilding the markdown content, which then causes `test-r-without-arrow` to
fail due to other dependencies
dongjoon-hyun commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-890016415
Thank you for your approval, @kiszk .
--
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
nealrichardson commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-890020555
Looks like you fixed one warning but there's still another:
https://github.com/ursacomputing/crossbow/runs/3203443397?check_suite_focus=true#step:7:192
--
This is an au
dongjoon-hyun commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-890020627
Thank you for your review and approval, @sunchao !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
kiszk commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-890023623
cc @emkornfield and @pitrou
--
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
rommelDB opened a new pull request #10843:
URL: https://github.com/apache/arrow/pull/10843
Changes:
- Added support for decimal128 and decimal256 json converted.
- Added unit tests for converting decimal128 and decimal256. (Note: This is
not providing inferring support)
--
This is
github-actions[bot] commented on pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#issuecomment-890023863
https://issues.apache.org/jira/browse/ARROW-4700
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
rommelDB commented on pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#issuecomment-890024085
Please take it a look @bkietz
--
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 t
emkornfield commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-890026317
As long as CI is passing 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 t
domoritz closed pull request #10825:
URL: https://github.com/apache/arrow/pull/10825
--
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-unsubs
domoritz closed pull request #10826:
URL: https://github.com/apache/arrow/pull/10826
--
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-unsubs
dongjoon-hyun commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-890027866
Thank you, @emkornfield . Yes, all CIs passed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
alamb commented on pull request #793:
URL: https://github.com/apache/arrow-datafusion/pull/793#issuecomment-890031194
Thanks for the reviews @Dandandan and @jhorstmann ! I plan to wait another
day or two to see if anyone else has feedback on this approach, but what I am
thinking of doing
sunchao closed pull request #10838:
URL: https://github.com/apache/arrow/pull/10838
--
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-unsubsc
nealrichardson commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-890037738
> > > I think building the things in `Suggests` has led to the build
rebuilding the markdown content, which then causes `test-r-without-arrow` to
fail due to other depend
sunchao commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-890039107
Merged! @emkornfield could you add @dongjoon-hyun to the Arrow contributor
list and assign the JIRA to him? thanks!
--
This is an automated message from the Apache Git Service
ElenaHenderson commented on a change in pull request #10667:
URL: https://github.com/apache/arrow/pull/10667#discussion_r680105559
##
File path: dev/conbench_envs/README.md
##
@@ -0,0 +1,30 @@
+
+## Benchmark Builds Env
+This directory contains
+- [benchmarks.env](../../dev/co
ElenaHenderson commented on a change in pull request #10667:
URL: https://github.com/apache/arrow/pull/10667#discussion_r680105559
##
File path: dev/conbench_envs/README.md
##
@@ -0,0 +1,30 @@
+
+## Benchmark Builds Env
+This directory contains
+- [benchmarks.env](../../dev/co
dongjoon-hyun commented on pull request #10838:
URL: https://github.com/apache/arrow/pull/10838#issuecomment-890044598
Thank you so much, @sunchao , @kiszk , @emkornfield !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
andygrove merged pull request #789:
URL: https://github.com/apache/arrow-datafusion/pull/789
--
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: githu
andygrove closed issue #47:
URL: https://github.com/apache/arrow-datafusion/issues/47
--
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-unsub
alamb commented on pull request #786:
URL: https://github.com/apache/arrow-datafusion/pull/786#issuecomment-890050571
This one seems like an incremental change towards #790 so I am going to
merge this in and rebase 790 so the changes are clearer.
--
This is an automated message from the
alamb merged pull request #786:
URL: https://github.com/apache/arrow-datafusion/pull/786
--
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-un
alamb closed issue #376:
URL: https://github.com/apache/arrow-datafusion/issues/376
--
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-unsubsc
alamb edited a comment on pull request #786:
URL: https://github.com/apache/arrow-datafusion/pull/786#issuecomment-890050571
This one seems like an incremental change towards #790 so I am going to
merge this in and rebase 790 so the changes are in that PR are smaller.
--
This is an auto
alamb commented on pull request #793:
URL: https://github.com/apache/arrow-datafusion/pull/793#issuecomment-890051505
Rebased given that https://github.com/apache/arrow-datafusion/pull/786 is
merged, so that this PR just shows the delta now
--
This is an automated message from the Apache
bkietz commented on a change in pull request #10667:
URL: https://github.com/apache/arrow/pull/10667#discussion_r679945382
##
File path: dev/conbench_envs/README.md
##
@@ -0,0 +1,209 @@
+
+# Benchmark Builds Env and Hooks
+This directory contains:
+- [benchmarks.env](benchmark
rommelDB edited a comment on pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#issuecomment-890024085
Please, take a look at it @bkietz
--
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
Dandandan commented on a change in pull request #793:
URL: https://github.com/apache/arrow-datafusion/pull/793#discussion_r680125826
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -508,7 +511,9 @@ fn dictionary_create_key_for_col(
}
/// Appends a sequenc
felipeblazing commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680124458
##
File path: cpp/src/arrow/json/converter_test.cc
##
@@ -96,5 +96,27 @@ TEST(ConverterTest, Timestamp) {
AssertConvert(timestamp(TimeUnit::SECON
andygrove commented on issue #771:
URL:
https://github.com/apache/arrow-datafusion/issues/771#issuecomment-890059076
Ballista is now at a point where it supports the TPC-H queries with
reasonable performance and I am now working on improving the developer
documentation and the user gu
felipeblazing commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680127350
##
File path: cpp/src/arrow/json/converter_test.cc
##
@@ -96,5 +96,27 @@ TEST(ConverterTest, Timestamp) {
AssertConvert(timestamp(TimeUnit::SECON
felipeblazing commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680128199
##
File path: cpp/src/arrow/json/converter.cc
##
@@ -147,6 +149,41 @@ class NumericConverter : public PrimitiveConverter {
const T& numeric_type_
felipeblazing commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680128769
##
File path: cpp/src/arrow/json/converter.cc
##
@@ -147,6 +149,41 @@ class NumericConverter : public PrimitiveConverter {
const T& numeric_type_
ElenaHenderson commented on a change in pull request #10667:
URL: https://github.com/apache/arrow/pull/10667#discussion_r680129299
##
File path: dev/conbench_envs/README.md
##
@@ -0,0 +1,30 @@
+
+## Benchmark Builds Env
+This directory contains
+- [benchmarks.env](../../dev/co
rommelDB commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680131459
##
File path: cpp/src/arrow/json/converter_test.cc
##
@@ -96,5 +96,27 @@ TEST(ConverterTest, Timestamp) {
AssertConvert(timestamp(TimeUnit::SECOND), s
rommelDB commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680139090
##
File path: cpp/src/arrow/json/converter.cc
##
@@ -147,6 +149,41 @@ class NumericConverter : public PrimitiveConverter {
const T& numeric_type_;
};
rommelDB commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680140494
##
File path: cpp/src/arrow/json/converter.cc
##
@@ -147,6 +149,41 @@ class NumericConverter : public PrimitiveConverter {
const T& numeric_type_;
};
bkietz commented on pull request #10722:
URL: https://github.com/apache/arrow/pull/10722#issuecomment-890076863
> -MakeScalarAggregateNode and MakeGroupByNode have quite different
function signatures, which makes working with the API confusing
> -GroupBy doesn't let you specify t
nirandaperera commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-890089863
@github-actions crossbow submit test-r-linux-as-cran
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
github-actions[bot] commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-890090342
Revision: 85fc16174a3acfecf24bd505aad5bbda69833111
Submitted crossbow builds: [ursacomputing/crossbow @
actions-680](https://github.com/ursacomputing/crossbow/
alamb commented on issue #640:
URL: https://github.com/apache/arrow-rs/issues/640#issuecomment-890099715
@waynexia i also run the verification script on mac without issue I think
the comment is outdated
--
This is an automated message from the Apache Git Service.
To respond to the mess
rommelDB commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680131459
##
File path: cpp/src/arrow/json/converter_test.cc
##
@@ -96,5 +96,27 @@ TEST(ConverterTest, Timestamp) {
AssertConvert(timestamp(TimeUnit::SECOND), s
pi-guy-in-the-sky opened a new issue #10844:
URL: https://github.com/apache/arrow/issues/10844
I'm currently working on a tool in C++ which needs to fetch data from an S3
bucket. Calling `arrow::fs::S3FileSystem::OpenInputStream` allows me to get an
`arrow::io::InputStream` that can read f
alamb commented on issue #640:
URL: https://github.com/apache/arrow-rs/issues/640#issuecomment-890101254
FWIW the question label already exists (I added it to this issue). It isn't
clear to me if everyone can add labels to issues in arrow-rs. Does it work for
you @waynexia ?
Perhaps
alamb opened a new issue #641:
URL: https://github.com/apache/arrow-rs/issues/641
**Describe the bug**
The statistics written by the arrow / parquet writer for String columns seem
to be incorrect.
**To Reproduce**
Run this code:
```rust
fn main() {
let inp
Dandandan merged pull request #637:
URL: https://github.com/apache/arrow-rs/pull/637
--
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-unsubs
Dandandan closed issue #636:
URL: https://github.com/apache/arrow-rs/issues/636
--
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...
alamb commented on a change in pull request #793:
URL: https://github.com/apache/arrow-datafusion/pull/793#discussion_r680188332
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -644,14 +645,63 @@ fn create_key_for_col(col: &ArrayRef, row: usize, vec:
&mut Ve
alamb commented on a change in pull request #793:
URL: https://github.com/apache/arrow-datafusion/pull/793#discussion_r680188812
##
File path: datafusion/src/scalar.rs
##
@@ -1347,6 +1545,29 @@ mod tests {
"{}", result);
}
+#[test]
+fn scalar_try
nirandaperera commented on pull request #10833:
URL: https://github.com/apache/arrow/pull/10833#issuecomment-890123629
@nealrichardson I think this is fine now! :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
nirandaperera opened a new pull request #10845:
URL: https://github.com/apache/arrow/pull/10845
Adding prelim impl of semi joins
--
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-actions[bot] commented on pull request #10845:
URL: https://github.com/apache/arrow/pull/10845#issuecomment-890126576
https://issues.apache.org/jira/browse/ARROW-13268
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
codecov-commenter edited a comment on pull request #628:
URL: https://github.com/apache/arrow-rs/pull/628#issuecomment-888530892
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/628?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
alamb commented on a change in pull request #792:
URL: https://github.com/apache/arrow-datafusion/pull/792#discussion_r680194021
##
File path: datafusion/src/execution/context.rs
##
@@ -691,6 +693,7 @@ impl Default for ExecutionConfig {
Arc::new(SimplifyExpress
bkietz closed pull request #10667:
URL: https://github.com/apache/arrow/pull/10667
--
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
westonpace commented on a change in pull request #10802:
URL: https://github.com/apache/arrow/pull/10802#discussion_r680168763
##
File path: cpp/src/arrow/compute/kernels/vector_selection.cc
##
@@ -1988,7 +1989,7 @@ class FilterMetaFunction : public MetaFunction {
Result> Ta
bkietz commented on a change in pull request #10843:
URL: https://github.com/apache/arrow/pull/10843#discussion_r680213131
##
File path: cpp/src/arrow/json/parser.cc
##
@@ -102,6 +102,8 @@ Status Kind::ForType(const DataType& type, Kind::type*
kind) {
Status Visit(const T
kou commented on pull request #10836:
URL: https://github.com/apache/arrow/pull/10836#issuecomment-890149852
+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.
To unsubscr
kou closed pull request #10836:
URL: https://github.com/apache/arrow/pull/10836
--
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...
gcca opened a new pull request #10846:
URL: https://github.com/apache/arrow/pull/10846
Add the super-scalar variant to set a bit.
--
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-actions[bot] commented on pull request #10846:
URL: https://github.com/apache/arrow/pull/10846#issuecomment-890159961
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 ope
Dandandan opened a new pull request #796:
URL: https://github.com/apache/arrow-datafusion/pull/796
# Which issue does this PR close?
Closes #.
# Rationale for this change
Move more unsupported conditions in LEFT join to filters.
This is needed to get TPC-
1 - 100 of 173053 matches
Mail list logo
