arthursunbao commented on issue #10885:
URL: https://github.com/apache/arrow/issues/10885#issuecomment-894037170
Hi westonpace,
Thanks for your quick response.
Our scenario is like this:
We have a recommendation system and we want to transfer the user data from
kafka
Dandandan commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683978762
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values =
Dandandan commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683978762
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values =
yjshen edited a comment on issue #824:
URL:
https://github.com/apache/arrow-datafusion/issues/824#issuecomment-894031550
If a singleton `context` is not the preferred way, is it possible to control
total memory usage for DataFusion across all physical operators?
- Should we extend
yjshen commented on issue #824:
URL:
https://github.com/apache/arrow-datafusion/issues/824#issuecomment-894031550
If a singleton `context` is not the preferred way, is it possible to control
total memory usage for DataFusion across all physical operators?
- Should we extend `Execut
JaguarPaw2409 commented on pull request #10450:
URL: https://github.com/apache/arrow/pull/10450#issuecomment-894030465
Is there any time frame to merge this request?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
Dandandan commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683975473
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values =
houqp commented on issue #825:
URL:
https://github.com/apache/arrow-datafusion/issues/825#issuecomment-894029329
seems like something that would be a good fit for design doc or user guide.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
Dandandan edited a comment on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-894021924
Some profiling output of the gby_null_new branch.
Dandandan edited a comment on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-894016110
Some good news, this is quite like mostly an improvement on the (more
challenging) db-benchmark aggregates.
Master:
```
q1 took 37 ms
q2 took 325 ms
Dandandan commented on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-894021924
Some profiling output of the gby_null_new branch.
houqp commented on issue #824:
URL:
https://github.com/apache/arrow-datafusion/issues/824#issuecomment-894021340
@yjshen based on the sample code in your io source PR, it looks like you
want the following API as a consumer of datafusion:
```rust
let ctx = ExecutionContext::get();
Dandandan commented on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-894016110
Some good news, this is quite like mostly an improvement on the (more
challenging) db-benchmark aggregates.
Master:
```
q1 took 37 ms
q2 took 325 ms
q3 to
Dandandan commented on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-894008907
Cool. I am doing some tests with db-benchmark, which includes some more
challenging queries.
--
This is an automated message from the Apache Git Service.
To respond to th
Dandandan commented on a change in pull request #827:
URL: https://github.com/apache/arrow-datafusion/pull/827#discussion_r683951579
##
File path: datafusion/src/physical_plan/hash_join.rs
##
@@ -476,18 +483,14 @@ fn update_hash(
// insert hashes to key of the hashmap
andygrove opened a new pull request #831:
URL: https://github.com/apache/arrow-datafusion/pull/831
# Which issue does this PR close?
Closes #830.
# Rationale for this change
Adds crate documentation that will appear in crates.io when the crates are
publishe
andygrove opened a new issue #830:
URL: https://github.com/apache/arrow-datafusion/issues/830
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
There is currently no crate documentation for the Ballista crates.
**Describe the
Dandandan commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683939523
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values =
cyb70289 commented on a change in pull request #10890:
URL: https://github.com/apache/arrow/pull/10890#discussion_r683931261
##
File path: cpp/src/arrow/compute/kernels/hash_aggregate.cc
##
@@ -1011,6 +1011,108 @@ struct GroupedSumFactory {
InputType argument_type;
};
+//
andygrove opened a new pull request #829:
URL: https://github.com/apache/arrow-datafusion/pull/829
# Which issue does this PR close?
Closes #828.
# Rationale for this change
Fixes docker build and the integration tests.
# What changes are included in
andygrove opened a new issue #828:
URL: https://github.com/apache/arrow-datafusion/issues/828
**Describe the bug**
I ran `./dev/integration-tests.sh` with latest from master just now and the
docker build phase failed with:
```
Step 20/52 : RUN cargo chef cook $RELEASE_FLAG --re
rok commented on pull request #10647:
URL: https://github.com/apache/arrow/pull/10647#issuecomment-893973246
@pitrou I refactored to have two kernel generators now - `MakeTemporal` and
`MakeSimpleUnaryTemporal`. Could you please review if this is dry enough now?
--
This is an automated m
aocsa commented on a change in pull request #10802:
URL: https://github.com/apache/arrow/pull/10802#discussion_r683908465
##
File path: cpp/src/arrow/compute/kernels/vector_selection.cc
##
@@ -2146,6 +2146,184 @@ class TakeMetaFunction : public MetaFunction {
}
};
+// ---
nealrichardson commented on pull request #10870:
URL: https://github.com/apache/arrow/pull/10870#issuecomment-893925233
Thanks for doing this! 🙏 In the future, it can be a good idea to grep the
source for the Jira issue number to see if there are any TODOs or skipped tests
related to the i
westonpace opened a new pull request #15:
URL: https://github.com/apache/arrow-cookbook/pull/15
Seems like a good idea
--
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 u
westonpace opened a new pull request #14:
URL: https://github.com/apache/arrow-cookbook/pull/14
Thsi gives `thisisnic` and `amol-` the ability to "assign, edit, and close
issues and pull requests". Given their help so far I think this should be
pretty straightforward.
--
This is an aut
westonpace commented on issue #9:
URL: https://github.com/apache/arrow-cookbook/issues/9#issuecomment-893873553
It should be pretty quick (I hope) to fix whatever we break. I guess I
don't know what the refresh rate is for the site publishing. I'll ask on Zulip.
--
This is an automated
westonpace commented on issue #9:
URL: https://github.com/apache/arrow-cookbook/issues/9#issuecomment-893873312
--
This is an automated message from the Apache Git Service.
westonpace opened a new issue #13:
URL: https://github.com/apache/arrow-cookbook/issues/13
Right now the CI build is pretty slow and a fair amount of time is spent
downloading / installing build dependencies and building Arrow. We should be
able to use prebuilt binaries for this.
T
thisisnic commented on issue #12:
URL: https://github.com/apache/arrow-cookbook/issues/12#issuecomment-893870552
My initial thoughts are that we should remove them otherwise it could send
out a confusing message to readers. I suppose as well, this is intrinsically
linked to the question o
alamb commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683834491
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values = grou
westonpace opened a new issue #12:
URL: https://github.com/apache/arrow-cookbook/issues/12
Related: Should we leave recipes around that are no longer valid at all on
the latest release of Arrow (e.g. the feature has been since removed,
presumably after being deprecated for some amount of t
westonpace closed issue #11:
URL: https://github.com/apache/arrow-cookbook/issues/11
--
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
westonpace opened a new issue #11:
URL: https://github.com/apache/arrow-cookbook/issues/11
I think there are a number of questions around Arrow versioning:
1. Should recipes be based on the latest released version of the
implementation? Or should they be based on the nightly build o
westonpace commented on issue #9:
URL: https://github.com/apache/arrow-cookbook/issues/9#issuecomment-893859299
Well, the docs for `.asf.yaml`
(https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features#Git.asf.yamlfeatures-Specifyingasub-directorytopublishto)
say you can u
thisisnic commented on issue #9:
URL: https://github.com/apache/arrow-cookbook/issues/9#issuecomment-893858271
That second URL certainly looks nicer, what kind of synchronization might be
needed?
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
thisisnic opened a new issue #10:
URL: https://github.com/apache/arrow-cookbook/issues/10
As more features are added to Apache Arrow, we might want to build versions
of the cookbook that are relevant to that release.
I'm not sure what a good strategy would be in terms of adding conte
westonpace opened a new issue #9:
URL: https://github.com/apache/arrow-cookbook/issues/9
It appears that in addition to gh-pages we can use Apache hosting. The only
real difference would be the URLs.
https://apache.github.io/arrow-cookbook
https://arrow.apache.org/cookbook
wesm commented on issue #8:
URL: https://github.com/apache/arrow-cookbook/issues/8#issuecomment-893844190
The main stream of e-mail notifications about commits or issues should match
up with what apache/arrow does
Regarding Zulip, I think you're referring to http://ursalabs.zulipchat
westonpace opened a new issue #8:
URL: https://github.com/apache/arrow-cookbook/issues/8
It appears for the ML it is a mere matter of updating `.asf.yaml`:
```
notifications:
commits: comm...@foo.apache.org
issues: iss...@foo.apache.org
pullrequests: d...@
westonpace merged pull request #7:
URL: https://github.com/apache/arrow-cookbook/pull/7
--
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-uns
kkraus14 commented on a change in pull request #10856:
URL: https://github.com/apache/arrow/pull/10856#discussion_r683802355
##
File path: format/ComputeIR.fbs
##
@@ -0,0 +1,521 @@
+/// Licensed to the Apache Software Foundation (ASF) under one
+/// or more contributor license
lidavidm closed pull request #10880:
URL: https://github.com/apache/arrow/pull/10880
--
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
lidavidm closed pull request #10870:
URL: https://github.com/apache/arrow/pull/10870
--
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 commented on a change in pull request #827:
URL: https://github.com/apache/arrow-datafusion/pull/827#discussion_r683788036
##
File path: datafusion/src/physical_plan/hash_join.rs
##
@@ -78,7 +81,14 @@ use log::debug;
// but the values don't match. Those are checked i
Dandandan commented on a change in pull request #827:
URL: https://github.com/apache/arrow-datafusion/pull/827#discussion_r683787741
##
File path: datafusion/src/physical_plan/hash_join.rs
##
@@ -476,18 +483,14 @@ fn update_hash(
// insert hashes to key of the hashmap
alamb commented on pull request #827:
URL: https://github.com/apache/arrow-datafusion/pull/827#issuecomment-893795473
I will review this carefully tomorrow
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
alamb commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683777504
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values = grou
kou commented on pull request #10033:
URL: https://github.com/apache/arrow/pull/10033#issuecomment-893779610
Thanks!
--
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 uns
alamb commented on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-893778745
I got enough of the approach described by @Dandandan in
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-893232614
working in https://github.com/apache/arrow-
kou commented on pull request #10865:
URL: https://github.com/apache/arrow/pull/10865#issuecomment-89393
I don't think that we need more i386 builds.
We can detect general i386 related build/test failures by the added Debian
task.
--
This is an automated message from the Apache Gi
github-actions[bot] commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893774630
Revision: 115afdc3c779847c2aed239c310001be94be35e2
Submitted crossbow builds: [ursacomputing/crossbow @
actions-726](https://github.com/ursacomputing/crossbow/
jonkeane commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893774131
@github-actions crossbow submit test-r-rhub-debian-gcc-devel-lto-latest
Still not totally general, but another try.
--
This is an automated message from the Apache Git
lidavidm commented on a change in pull request #10877:
URL: https://github.com/apache/arrow/pull/10877#discussion_r683758563
##
File path: cpp/src/arrow/filesystem/s3fs.cc
##
@@ -483,6 +484,29 @@ std::string FormatRange(int64_t start, int64_t length) {
return ss.str();
}
lidavidm commented on a change in pull request #10877:
URL: https://github.com/apache/arrow/pull/10877#discussion_r683758372
##
File path: cpp/src/arrow/filesystem/s3fs.cc
##
@@ -483,6 +484,29 @@ std::string FormatRange(int64_t start, int64_t length) {
return ss.str();
}
lidavidm commented on a change in pull request #10877:
URL: https://github.com/apache/arrow/pull/10877#discussion_r683756665
##
File path: cpp/src/arrow/filesystem/s3fs.h
##
@@ -69,6 +69,13 @@ enum class S3CredentialsKind : int8_t {
WebIdentity
};
+/// Pure virtual class
lidavidm commented on a change in pull request #10877:
URL: https://github.com/apache/arrow/pull/10877#discussion_r683755534
##
File path: cpp/src/arrow/filesystem/s3fs.h
##
@@ -69,6 +69,22 @@ enum class S3CredentialsKind : int8_t {
WebIdentity
};
+/// Pure virtual class
Dandandan commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683754338
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values =
Dandandan commented on a change in pull request #808:
URL: https://github.com/apache/arrow-datafusion/pull/808#discussion_r683752478
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -363,55 +348,74 @@ fn group_aggregate_batch(
let mut group_by_values =
kszucs commented on pull request #524:
URL: https://github.com/apache/arrow-datafusion/pull/524#issuecomment-893760912
@alamb this should be good to go, though we should revisit the FFI bindings
in arrow-rs and a potential `arrow-rs <-> pyarrow` bridge implemented in
arrow-rs in the future
github-actions[bot] commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-893760223
Revision: 69ff20b2afba02b7ddfde316234a61403771959a
Submitted crossbow builds: [ursacomputing/crossbow @
actions-725](https://github.com/ursacomputing/crossbow/
jonkeane commented on pull request #10841:
URL: https://github.com/apache/arrow/pull/10841#issuecomment-893759682
@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 g
github-actions[bot] commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893756286
Revision: 6d4f79567f6bda7ec0e21e3ae772143448cb583e
Submitted crossbow builds: [ursacomputing/crossbow @
actions-724](https://github.com/ursacomputing/crossbow/
jonkeane commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893755892
@github-actions crossbow submit test-r-rhub-debian-gcc-devel-lto-latest
This isn't a complete solution (i.e. it likely will not work on non-lto
builds), but should be eno
github-actions[bot] removed a comment on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893755475
Revision: e0396dc7b609f01d3f95ad875c58bcccf5e03181
Submitted crossbow builds: [ursacomputing/crossbow @
actions-723](https://github.com/ursacomputing/c
github-actions[bot] commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893755475
Revision: e0396dc7b609f01d3f95ad875c58bcccf5e03181
Submitted crossbow builds: [ursacomputing/crossbow @
actions-723](https://github.com/ursacomputing/crossbow/
jonkeane removed a comment on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893754857
@github-actions crossbow submit test-r-rhub-debian-gcc-devel-lto-latest
This is not a complete solution (e.g. won't work well on non-LTO platforms),
but should te
jonkeane commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893754857
@github-actions crossbow submit test-r-rhub-debian-gcc-devel-lto-latest
This is not a complete solution (e.g. won't work well on non-LTO platforms),
but should test if we
nealrichardson commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893727566
Ok, that builds and links but segfaults on load. @jonkeane could you add
back in here your cmake changes (I think you force-pushed them out of your
previous branch) and t
lidavidm commented on a change in pull request #10877:
URL: https://github.com/apache/arrow/pull/10877#discussion_r683730535
##
File path: cpp/src/arrow/filesystem/s3fs.h
##
@@ -69,6 +69,13 @@ enum class S3CredentialsKind : int8_t {
WebIdentity
};
+/// Pure virtual class
neil-b commented on a change in pull request #10877:
URL: https://github.com/apache/arrow/pull/10877#discussion_r683729032
##
File path: cpp/src/arrow/filesystem/s3fs.h
##
@@ -69,6 +69,13 @@ enum class S3CredentialsKind : int8_t {
WebIdentity
};
+/// Pure virtual class fo
github-actions[bot] commented on pull request #10890:
URL: https://github.com/apache/arrow/pull/10890#issuecomment-893724407
https://issues.apache.org/jira/browse/ARROW-13575
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
alamb commented on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-893712779
@Dandandan I used your approach in
https://github.com/apache/arrow-datafusion/pull/808/commits/9ad6719155932e69e211e3f90cf5e4beb0bdc0ad
and it seems to have worked. I want to
westonpace commented on issue #10885:
URL: https://github.com/apache/arrow/issues/10885#issuecomment-893706079
Can you expand a little bit on what problem you are trying to solve? There
is a schema. It can be serialized to parquet and to the Arrow IPC format. It
specifies the name and d
github-actions[bot] commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893703559
Revision: e0396dc7b609f01d3f95ad875c58bcccf5e03181
Submitted crossbow builds: [ursacomputing/crossbow @
actions-722](https://github.com/ursacomputing/crossbow/
github-actions[bot] commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893701879
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
nealrichardson commented on pull request #10889:
URL: https://github.com/apache/arrow/pull/10889#issuecomment-893701978
@github-actions crossbow submit test-r-rhub-debian-gcc-devel-lto-latest
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
westonpace closed pull request #10882:
URL: https://github.com/apache/arrow/pull/10882
--
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-unsu
westonpace commented on pull request #10882:
URL: https://github.com/apache/arrow/pull/10882#issuecomment-893696362
Ah, please forgive my foolishness. I'll reopen if I find the root cause but
now I'm wondering if I was just testing with R code built against an older
version of Arrow.
--
jorgecarleitao commented on issue #824:
URL:
https://github.com/apache/arrow-datafusion/issues/824#issuecomment-893683156
Thanks for the suggestion.
I do not think we should do this in `DataFusion`. These use-cases imo should
addressed by consumers of DataFusion that decide how they
nealrichardson commented on pull request #10710:
URL: https://github.com/apache/arrow/pull/10710#issuecomment-893681346
@jonkeane can you help make sure that this isn't breaking any builds?
crossbow keeps returning failures, I haven't had the chance to examine why.
This PR shouldn't be cha
bkietz commented on a change in pull request #10431:
URL: https://github.com/apache/arrow/pull/10431#discussion_r683685887
##
File path: cpp/src/arrow/dataset/file_skyhook.h
##
@@ -0,0 +1,275 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contribu
nealrichardson commented on a change in pull request #10888:
URL: https://github.com/apache/arrow/pull/10888#discussion_r683683809
##
File path: r/R/dataset-scan.R
##
@@ -85,10 +85,22 @@ Scanner$create <- function(dataset,
# To handle mutate() on Table/RecordBatch, we ne
nealrichardson commented on a change in pull request #10888:
URL: https://github.com/apache/arrow/pull/10888#discussion_r683681911
##
File path: r/R/dataset-scan.R
##
@@ -85,10 +85,22 @@ Scanner$create <- function(dataset,
# To handle mutate() on Table/RecordBatch, we ne
github-actions[bot] commented on pull request #10888:
URL: https://github.com/apache/arrow/pull/10888#issuecomment-893669493
https://issues.apache.org/jira/browse/ARROW-13560
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
nealrichardson closed pull request #10829:
URL: https://github.com/apache/arrow/pull/10829
--
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-
alamb commented on issue #790:
URL:
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-893661708
> I put an example of the latest suggestion here:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=9157e7ae2ad4d84f8bd6c358c42722cb
That is very co
github-actions[bot] commented on pull request #10887:
URL: https://github.com/apache/arrow/pull/10887#issuecomment-893659166
https://issues.apache.org/jira/browse/ARROW-13311
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
lidavidm commented on pull request #10870:
URL: https://github.com/apache/arrow/pull/10870#issuecomment-893658081
I think the failures here are flukes (GHA seems to be having a bit of
trouble), let's see when the Travis build finishes.
--
This is an automated message from the Apache Git
lidavidm commented on pull request #10880:
URL: https://github.com/apache/arrow/pull/10880#issuecomment-893657720
The AppVeyor failure looks like a flake (s3fs on windows being flaky). I
kicked off the 'Dev PR' pipeline again though I expect that to also be a fluke.
--
This is an automat
lidavidm opened a new pull request #10887:
URL: https://github.com/apache/arrow/pull/10887
This adds a section about hash aggregates. Also, adds Kleene logic to the
hash any/all kernels.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
lidavidm commented on pull request #10887:
URL: https://github.com/apache/arrow/pull/10887#issuecomment-893657014
--
This is an automated message from the Apache Git Service.
alamb commented on pull request #719:
URL: https://github.com/apache/arrow-datafusion/pull/719#issuecomment-893655863
I will plan to merge this PR once the CI tests pass
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
alamb commented on a change in pull request #811:
URL: https://github.com/apache/arrow-datafusion/pull/811#discussion_r683659016
##
File path: datafusion/src/execution/context.rs
##
@@ -125,12 +127,26 @@ pub struct ExecutionContext {
pub state: Arc>,
}
+lazy_static! {
alamb merged pull request #797:
URL: https://github.com/apache/arrow-datafusion/pull/797
--
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 commented on pull request #797:
URL: https://github.com/apache/arrow-datafusion/pull/797#issuecomment-893654846
Thanks again @seddonm1
--
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
romgrk-comparative commented on issue #10803:
URL: https://github.com/apache/arrow/issues/10803#issuecomment-893651302
Alright, yes that was `DictionaryArray`, thanks for the precisions!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
romgrk-comparative closed issue #10803:
URL: https://github.com/apache/arrow/issues/10803
--
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-u
Dandandan opened a new pull request #827:
URL: https://github.com/apache/arrow-datafusion/pull/827
# Which issue does this PR close?
Closes #826
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing change
aucahuasi commented on pull request #10880:
URL: https://github.com/apache/arrow/pull/10880#issuecomment-893634764
> LGTM. I left a suggestion for the tests since we have some helper
functions to make those cases easier to write.
I applied the suggestions, but let me know if I forgot
1 - 100 of 337 matches
Mail list logo
