Dandandan commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635803983
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
Dandandan commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635802863
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
Dandandan commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635802863
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
emkornfield commented on a change in pull request #10201:
URL: https://github.com/apache/arrow/pull/10201#discussion_r635792044
##
File path: cpp/src/jni/dataset/jni_wrapper.cc
##
@@ -166,6 +156,126 @@ class DisposableScannerAdaptor {
}
};
+/// \brief Simple scan task imp
emkornfield commented on a change in pull request #10201:
URL: https://github.com/apache/arrow/pull/10201#discussion_r635791186
##
File path: cpp/src/jni/dataset/jni_util.cc
##
@@ -211,32 +250,183 @@ std::vector ToStringVector(JNIEnv* env,
jobjectArray& str_array) {
return
emkornfield commented on a change in pull request #10201:
URL: https://github.com/apache/arrow/pull/10201#discussion_r635789662
##
File path: cpp/src/jni/dataset/jni_util.cc
##
@@ -162,31 +192,40 @@ std::shared_ptr
ReservationListenableMemoryPool::get_listen
ReservationList
emkornfield commented on a change in pull request #10201:
URL: https://github.com/apache/arrow/pull/10201#discussion_r635788017
##
File path: cpp/src/jni/dataset/jni_util.cc
##
@@ -16,36 +16,44 @@
// under the License.
#include "jni/dataset/jni_util.h"
-
+#include "arrow/ip
emkornfield commented on a change in pull request #10201:
URL: https://github.com/apache/arrow/pull/10201#discussion_r635787880
##
File path: cpp/src/jni/dataset/jni_util.cc
##
@@ -16,36 +16,44 @@
// under the License.
#include "jni/dataset/jni_util.h"
-
+#include "arrow/ip
emkornfield commented on pull request #10308:
URL: https://github.com/apache/arrow/pull/10308#issuecomment-844723885
It looks like this might be failing tests on bigendian machines (see CI).
Also can we separate C++ and JNI compoments into two separate issues/PRs.
--
This is an automate
emkornfield commented on issue #10187:
URL: https://github.com/apache/arrow/issues/10187#issuecomment-844721791
CC @trxcllnt @TheNeuralBit does this sound like a bug?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
emkornfield closed pull request #9964:
URL: https://github.com/apache/arrow/pull/9964
--
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
emkornfield commented on pull request #9964:
URL: https://github.com/apache/arrow/pull/9964#issuecomment-844720874
Sorry looks good. Will merge.
--
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
emkornfield closed pull request #10071:
URL: https://github.com/apache/arrow/pull/10071
--
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
praveenbingo closed pull request #9724:
URL: https://github.com/apache/arrow/pull/9724
--
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, ple
praveenbingo closed pull request #10218:
URL: https://github.com/apache/arrow/pull/10218
--
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
praveenbingo closed pull request #9861:
URL: https://github.com/apache/arrow/pull/9861
--
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, ple
praveenbingo closed pull request #10095:
URL: https://github.com/apache/arrow/pull/10095
--
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
domoritz commented on pull request #10340:
URL: https://github.com/apache/arrow/pull/10340#issuecomment-844690061
Done. This should be ready to go.
--
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
domoritz edited a comment on pull request #10361:
URL: https://github.com/apache/arrow/pull/10361#issuecomment-844678159
* [x] actually load the sources by default
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
github-actions[bot] commented on pull request #10362:
URL: https://github.com/apache/arrow/pull/10362#issuecomment-844688019
https://issues.apache.org/jira/browse/ARROW-12833
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
domoritz opened a new pull request #10362:
URL: https://github.com/apache/arrow/pull/10362
--
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,
github-actions[bot] commented on pull request #10355:
URL: https://github.com/apache/arrow/pull/10355#issuecomment-844678752
Revision: cb827c5eb69e66491bdbbdf8339414556bfd346a
Submitted crossbow builds: [ursacomputing/crossbow @
actions-426](https://github.com/ursacomputing/crossbow/
kou commented on pull request #10355:
URL: https://github.com/apache/arrow/pull/10355#issuecomment-844678347
@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
domoritz commented on pull request #10361:
URL: https://github.com/apache/arrow/pull/10361#issuecomment-844678159
* [ ] actually load the sources by default
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
ursabot edited a comment on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844625395
Benchmark runs are scheduled for baseline =
3a85ab482c3181f207c68c52a66f240da49e8995 and contender =
aa9b15936d54dcf09ee1857e04515ec21698fa93. Results will be available a
westonpace commented on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844672916
The EC2 instances where we really want the benchmarks are running out of
memory it seems. I was able to get some times from ursa-i9-9960x. Note, it
does not line up the ben
github-actions[bot] commented on pull request #10361:
URL: https://github.com/apache/arrow/pull/10361#issuecomment-844659697
https://issues.apache.org/jira/browse/ARROW-12832
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
domoritz opened a new pull request #10361:
URL: https://github.com/apache/arrow/pull/10361
--
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,
ursabot edited a comment on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844625395
Benchmark runs are scheduled for baseline =
3a85ab482c3181f207c68c52a66f240da49e8995 and contender =
aa9b15936d54dcf09ee1857e04515ec21698fa93. Results will be available a
domoritz commented on pull request #10360:
URL: https://github.com/apache/arrow/pull/10360#issuecomment-844643087
Merge after https://github.com/apache/arrow/pull/10340
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
github-actions[bot] commented on pull request #10360:
URL: https://github.com/apache/arrow/pull/10360#issuecomment-844641209
https://issues.apache.org/jira/browse/ARROW-12808
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
github-actions[bot] commented on pull request #10360:
URL: https://github.com/apache/arrow/pull/10360#issuecomment-844639263
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
domoritz opened a new pull request #10360:
URL: https://github.com/apache/arrow/pull/10360
--
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,
ursabot edited a comment on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844625395
Benchmark runs are scheduled for baseline =
3a85ab482c3181f207c68c52a66f240da49e8995 and contender =
aa9b15936d54dcf09ee1857e04515ec21698fa93. Results will be available a
kou commented on pull request #10355:
URL: https://github.com/apache/arrow/pull/10355#issuecomment-844632661
OK. Let's try `gaXXX_`. If `gaXXX_` conflicts with other products, we can
rethink this.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
kou commented on pull request #9758:
URL: https://github.com/apache/arrow/pull/9758#issuecomment-844630788
@nealrichardson Could you confirm suggestions from @rok ? Can we merge this
with these suggestions?
--
This is an automated message from the Apache Git Service.
To respond to the me
kou closed pull request #10359:
URL: https://github.com/apache/arrow/pull/10359
--
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
kou commented on pull request #10359:
URL: https://github.com/apache/arrow/pull/10359#issuecomment-844629390
+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
ursabot commented on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844625395
Benchmark runs are scheduled for baseline =
3a85ab482c3181f207c68c52a66f240da49e8995 and contender =
aa9b15936d54dcf09ee1857e04515ec21698fa93. Results will be available as each
kou commented on pull request #10359:
URL: https://github.com/apache/arrow/pull/10359#issuecomment-844613722
macOS jobs were timed out on master:
* https://github.com/apache/arrow/runs/2603100840
* https://github.com/apache/arrow/runs/2603100814
* https://github.com/apache/arrow
westonpace commented on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844625141
@ursabot please benchmark lang=Python
--
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 t
kou opened a new pull request #10359:
URL: https://github.com/apache/arrow/pull/10359
This was introduced by #9119 (ARROW-1152) but this is no longer
needed.
--
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-actions[bot] commented on pull request #10359:
URL: https://github.com/apache/arrow/pull/10359#issuecomment-844613209
https://issues.apache.org/jira/browse/ARROW-12831
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
cyb70289 closed pull request #10274:
URL: https://github.com/apache/arrow/pull/10274
--
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, pleas
Jimexist commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635676450
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
Jimexist commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635675124
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
Jimexist commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635673607
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
Jimexist commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635672736
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
ianmcook commented on pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#issuecomment-844548176
What to do about the options class might be awkward. You could subclass
`MatchSubstringOptions`, but I think ultimately we might incorporate an option
for case insensitivity in
lidavidm commented on pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#issuecomment-844542476
No, but in that case we'd just not dispatch to that path.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
ianmcook commented on pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#issuecomment-844542150
But can `PlainSubstringMatcher` do case-insensitive matching?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
lidavidm commented on pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#issuecomment-844540228
We could easily add a flag to toggle insensitivity in RE2 as well. (Would be
better than allocating a new string and potentially dealing with Unicode.)
--
This is an automate
ianmcook commented on pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#issuecomment-844539780
P.S. case insensitivity can be achieved by using one of the `_upper` or
`_lower` kernels, which I think is good enough
--
This is an automated message from the Apache Git Ser
michalursa commented on pull request #9768:
URL: https://github.com/apache/arrow/pull/9768#issuecomment-844533147
> > @michalursa looks like tests are hanging on our [bigendian
CI](https://travis-ci.com/github/apache/arrow/jobs/506281680#L2907). Is this
quick to address or should we leave
lidavidm commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635613255
##
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##
@@ -494,6 +494,95 @@ const FunctionDoc match_substring_regex_doc(
"position.\n"
ianmcook commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635609498
##
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##
@@ -494,6 +494,95 @@ const FunctionDoc match_substring_regex_doc(
"position.\n"
lidavidm commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635602560
##
File path: docs/source/cpp/compute.rst
##
@@ -529,28 +529,36 @@ Containment tests
+---++
ianmcook commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635601185
##
File path: docs/source/cpp/compute.rst
##
@@ -529,28 +529,36 @@ Containment tests
+---++
lidavidm commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635598720
##
File path: docs/source/cpp/compute.rst
##
@@ -529,28 +529,36 @@ Containment tests
+---++
ianmcook commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635595821
##
File path: docs/source/cpp/compute.rst
##
@@ -529,28 +529,36 @@ Containment tests
+---++
augustoasilva commented on a change in pull request #10059:
URL: https://github.com/apache/arrow/pull/10059#discussion_r635595413
##
File path: cpp/src/gandiva/replace_holder.h
##
@@ -0,0 +1,97 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contri
ianmcook commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635594889
##
File path: docs/source/cpp/compute.rst
##
@@ -529,28 +529,36 @@ Containment tests
+---++
lidavidm commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635582443
##
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##
@@ -494,6 +494,93 @@ const FunctionDoc match_substring_regex_doc(
"position.\n"
lidavidm commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635582104
##
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##
@@ -494,6 +494,93 @@ const FunctionDoc match_substring_regex_doc(
"position.\n"
nealrichardson commented on a change in pull request #10269:
URL: https://github.com/apache/arrow/pull/10269#discussion_r635572831
##
File path: r/R/table.R
##
@@ -175,12 +175,17 @@ Table$create <- function(..., schema = NULL) {
return(dplyr::group_by(out, !!!dplyr::groups
ianmcook commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635580566
##
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##
@@ -494,6 +494,93 @@ const FunctionDoc match_substring_regex_doc(
"position.\n"
ianmcook commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635580566
##
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##
@@ -494,6 +494,93 @@ const FunctionDoc match_substring_regex_doc(
"position.\n"
jorgecarleitao commented on pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#issuecomment-844480669
```
cargo test --lib
...
Finished dev [unoptimized + debuginfo] target(s)
```
🎉
I temporarily deactivated the predicate pushdown for parquet,
ianmcook commented on a change in pull request #10356:
URL: https://github.com/apache/arrow/pull/10356#discussion_r635579350
##
File path: cpp/src/arrow/compute/kernels/scalar_string.cc
##
@@ -494,6 +494,93 @@ const FunctionDoc match_substring_regex_doc(
"position.\n"
nealrichardson commented on pull request #9768:
URL: https://github.com/apache/arrow/pull/9768#issuecomment-844456233
> @michalursa looks like tests are hanging on our [bigendian
CI](https://travis-ci.com/github/apache/arrow/jobs/506281680#L2907). Is this
quick to address or should we leav
Dandandan commented on pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#issuecomment-84016
I also think this is looking good. I think we might clean things up a bit
while touching the code, I added some suggestions.
--
This is an automated message from the
Dandandan commented on a change in pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635557126
##
File path: datafusion/src/physical_optimizer/pruning.rs
##
@@ -0,0 +1,769 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+/
Dandandan commented on a change in pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635554150
##
File path: datafusion/src/physical_optimizer/pruning.rs
##
@@ -0,0 +1,769 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+/
Dandandan commented on a change in pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635554027
##
File path: datafusion/src/physical_optimizer/pruning.rs
##
@@ -0,0 +1,769 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+/
Dandandan commented on a change in pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635553323
##
File path: datafusion/src/physical_optimizer/pruning.rs
##
@@ -0,0 +1,769 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+/
Dandandan commented on a change in pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635552829
##
File path: datafusion/src/physical_optimizer/pruning.rs
##
@@ -0,0 +1,769 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+/
alamb commented on pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#issuecomment-844432321
FYI @yordan-pavlov and @returnString
--
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
github-actions[bot] commented on pull request #10345:
URL: https://github.com/apache/arrow/pull/10345#issuecomment-844429851
Revision: 63e78da4059a266c6dc7cba8cfe73ef60ed4a4e5
Submitted crossbow builds: [ursacomputing/crossbow @
actions-425](https://github.com/ursacomputing/crossbow/
jonkeane commented on pull request #10345:
URL: https://github.com/apache/arrow/pull/10345#issuecomment-844428125
@github-actions crossbow submit revdep-r-check
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
nealrichardson commented on a change in pull request #10334:
URL: https://github.com/apache/arrow/pull/10334#discussion_r635544090
##
File path: r/R/compute.R
##
@@ -286,3 +286,8 @@ cast_options <- function(safe = TRUE, ...) {
)
modifyList(opts, list(...))
}
+
+strptime_
lidavidm commented on a change in pull request #10321:
URL: https://github.com/apache/arrow/pull/10321#discussion_r635534220
##
File path: cpp/src/arrow/csv/parser_test.cc
##
@@ -536,22 +537,42 @@ TEST(BlockParser, QuotesSpecial) {
TEST(BlockParser, MismatchingNumColumns) {
jduo commented on a change in pull request #9368:
URL: https://github.com/apache/arrow/pull/9368#discussion_r635534750
##
File path:
java/flight/flight-sql/src/main/java/org/apache/arrow/flight/sql/FlightSQLClientUtils.java
##
@@ -0,0 +1,219 @@
+/*
+ * Licensed to the Apache S
sunchao commented on a change in pull request #307:
URL: https://github.com/apache/arrow-rs/pull/307#discussion_r635532034
##
File path: parquet/src/column/writer.rs
##
@@ -607,9 +607,11 @@ impl ColumnWriterImpl {
let max_def_level = self.descr.max_def_level();
ursabot edited a comment on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844381307
Benchmark runs are scheduled for baseline =
aa37d197a63a7efbc0660f9cea2f75cc08c30587 and contender =
fb02d17a77900b4012c3c57f865e68cda0a41274. Results will be available a
Dandandan edited a comment on issue #364:
URL:
https://github.com/apache/arrow-datafusion/issues/364#issuecomment-844401489
I agree combining them makes a lot of sense :+1:
In aggregates I was also stumbling on some strange issues relating to
evaluation of scalars while working on
ursabot edited a comment on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844381307
Benchmark runs are scheduled for baseline =
aa37d197a63a7efbc0660f9cea2f75cc08c30587 and contender =
fb02d17a77900b4012c3c57f865e68cda0a41274. Results will be available a
github-actions[bot] commented on pull request #10358:
URL: https://github.com/apache/arrow/pull/10358#issuecomment-844401662
https://issues.apache.org/jira/browse/ARROW-2665
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
Dandandan commented on issue #364:
URL:
https://github.com/apache/arrow-datafusion/issues/364#issuecomment-844401489
I agree combining them makes a lot of sense :+1:
In aggregates I was also stumbling on some strange issues relating to
evaluation of scalars while working on this:
ursabot edited a comment on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844381307
Benchmark runs are scheduled for baseline =
aa37d197a63a7efbc0660f9cea2f75cc08c30587 and contender =
fb02d17a77900b4012c3c57f865e68cda0a41274. Results will be available a
bkietz commented on a change in pull request #10296:
URL: https://github.com/apache/arrow/pull/10296#discussion_r635520473
##
File path: docs/source/cpp/authoring_compute_functions.rst
##
@@ -0,0 +1,421 @@
+.. Licensed to the Apache Software Foundation (ASF) under one
+.. or mo
ursabot edited a comment on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844381307
Benchmark runs are scheduled for baseline =
aa37d197a63a7efbc0660f9cea2f75cc08c30587 and contender =
fb02d17a77900b4012c3c57f865e68cda0a41274. Results will be available a
Dandandan commented on a change in pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#discussion_r635518750
##
File path: ballista/rust/core/proto/ballista.proto
##
@@ -151,6 +153,25 @@ message AggregateExprNode {
LogicalExprNode expr = 2;
}
+enum
codecov-commenter commented on pull request #365:
URL: https://github.com/apache/arrow-datafusion/pull/365#issuecomment-844395540
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/365?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comment
codecov-commenter edited a comment on pull request #366:
URL: https://github.com/apache/arrow-datafusion/pull/366#issuecomment-844390334
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/366?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+
bkietz commented on a change in pull request #10296:
URL: https://github.com/apache/arrow/pull/10296#discussion_r633591259
##
File path: docs/source/cpp/authoring_compute_functions.rst
##
@@ -0,0 +1,421 @@
+.. Licensed to the Apache Software Foundation (ASF) under one
+.. or mo
lidavidm opened a new pull request #10358:
URL: https://github.com/apache/arrow/pull/10358
Add a simple index() kernel.
Note that the Python start/end options are handled entirely in Python, not
in the kernel itself.
Short-circuiting is somewhat implemented: the kernel executo
codecov-commenter commented on pull request #366:
URL: https://github.com/apache/arrow-datafusion/pull/366#issuecomment-844390334
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/366?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comment
alamb commented on pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#issuecomment-844384440
I will try to review this later today but I may not get to it until tomorrow
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
ursabot commented on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844381307
Benchmark runs are scheduled for baseline =
aa37d197a63a7efbc0660f9cea2f75cc08c30587 and contender =
fb02d17a77900b4012c3c57f865e68cda0a41274. Results will be available as each
westonpace commented on pull request #10118:
URL: https://github.com/apache/arrow/pull/10118#issuecomment-844381092
@ursabot please benchmark lang=Python
--
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 t
1 - 100 of 194 matches
Mail list logo
