jonathanc-n closed pull request #16210: feat: Support null aware + equijoins
for `NestedLoopJoin`
URL: https://github.com/apache/datafusion/pull/16210
--
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 PR #15770:
URL: https://github.com/apache/datafusion/pull/15770#issuecomment-2974698767
> @Dandandan thank you so much for pushing this forward! Really appreciate
the help and collaboration.
>
> Since you've been looking at the code do you have any thoughts on [
adriangb commented on PR #15770:
URL: https://github.com/apache/datafusion/pull/15770#issuecomment-2974704013
> I think two different phases sounds good.
>
> I don't really like complicating FilterPushdown with post / pre "phases".
Why not creating the `PushDownDynamicFilters` one
Dandandan commented on PR #16249:
URL: https://github.com/apache/datafusion/pull/16249#issuecomment-2974721937
This is for me
```diff
--- i/datafusion/physical-plan/src/coalesce/mod.rs
+++ w/datafusion/physical-plan/src/coalesce/mod.rs
@@ -33,6 +33,9 @@ pub struct LimitedBatc
Huy1Ng opened a new pull request, #1273:
URL: https://github.com/apache/datafusion-ballista/pull/1273
# Which issue does this PR close?
Closes #1117.
# Rationale for this change
Tests failed on Windows image because the path is not resolved/canonicalized
before feeding into
mmooyyii commented on issue #16406:
URL: https://github.com/apache/datafusion/issues/16406#issuecomment-2974920966
```
use bigdecimal::num_bigint::BigInt;
use bigdecimal::{BigDecimal};
use std::fs;
fn main() {
let f = fs::read_to_string("/tmp/decimal.csv").unwrap();
Huy1Ng commented on issue #1117:
URL:
https://github.com/apache/datafusion-ballista/issues/1117#issuecomment-2974921643
I made a PR to fix this bug here:
https://github.com/apache/datafusion-ballista/pull/1273
--
This is an automated message from the Apache Git Service.
To respond to the
github-actions[bot] commented on PR #15683:
URL: https://github.com/apache/datafusion/pull/15683#issuecomment-2974908011
Thank you for your contribution. Unfortunately, this pull request is stale
because it has been open 60 days with no activity. Please remove the stale
label or comment or
github-actions[bot] commented on PR #15482:
URL: https://github.com/apache/datafusion/pull/15482#issuecomment-2974908124
Thank you for your contribution. Unfortunately, this pull request is stale
because it has been open 60 days with no activity. Please remove the stale
label or comment or
Chen-Yuan-Lai commented on code in PR #16324:
URL: https://github.com/apache/datafusion/pull/16324#discussion_r2148615171
##
datafusion/core/tests/sql/explain_analyze.rs:
##
@@ -797,14 +796,16 @@ async fn explain_physical_plan_only() {
let sql = "EXPLAIN select count(*) fro
Dandandan commented on PR #15770:
URL: https://github.com/apache/datafusion/pull/15770#issuecomment-2974699280
Apart from that, I agree with the sentiment to try eliminate it from
`EnforceSorting`
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
mmooyyii commented on issue #16406:
URL: https://github.com/apache/datafusion/issues/16406#issuecomment-2975198860
https://github.com/apache/datafusion/blob/ca0b760af6137c0dbec8b07daa5f48e262420cb5/datafusion/functions-aggregate/src/sum.rs#L309
Use `*v = v.add_checked(x)?;` ?
--
alamb commented on PR #16409:
URL: https://github.com/apache/datafusion/pull/16409#issuecomment-2973641672
> I'm personally intrigued tbh but I'd say the DF core should be agnostic of
specific data-driven architecture(like Spark) even if we do a lot of Spark
integration like Sail or Comet.
alamb commented on code in PR #16409:
URL: https://github.com/apache/datafusion/pull/16409#discussion_r2147518786
##
datafusion/sqllogictest/test_files/spark/array/array.slt:
##
@@ -0,0 +1,22 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
xudong963 commented on PR #16417:
URL: https://github.com/apache/datafusion/pull/16417#issuecomment-2973660882
https://github.com/user-attachments/assets/42ef5941-3879-4b2a-84de-51162c2f06fb";
/>
I don't find any changes
--
This is an automated message from the Apache Git Service.
xudong963 commented on issue #15885:
URL: https://github.com/apache/datafusion/issues/15885#issuecomment-2973683721
FYI, I just followed the latest paper from TUM, "Improving Unnesting of
Complex Queries", and will learn about the current code in DF and read the
latest PRs in DF about unnes
jonmmease opened a new pull request, #16418:
URL: https://github.com/apache/datafusion/pull/16418
## Summary
This PR fixes the WASM compilation issue (#13513) by making sqlparser's
`recursive-protection` feature optional. This allows DataFusion to be compiled
for WebAssembly targets
alamb commented on code in PR #16399:
URL: https://github.com/apache/datafusion/pull/16399#discussion_r2147731773
##
docs/source/contributor-guide/roadmap.md:
##
@@ -46,81 +46,12 @@ make review efficient and avoid surprises.
# Quarterly Roadmap
-A quarterly roadmap will be
alamb commented on PR #16417:
URL: https://github.com/apache/datafusion/pull/16417#issuecomment-2973734192
> https://private-user-images.githubusercontent.com/41979257/455251135-42ef5941-3879-4b2a-84de-51162c2f06fb.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkI
zhuqi-lucas commented on code in PR #16395:
URL: https://github.com/apache/datafusion/pull/16395#discussion_r2148133565
##
datafusion-examples/examples/embedding_parquet_indexes.rs:
##
@@ -0,0 +1,243 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
zhuqi-lucas commented on code in PR #16395:
URL: https://github.com/apache/datafusion/pull/16395#discussion_r2148132009
##
datafusion-examples/examples/embedding_parquet_indexes.rs:
##
@@ -0,0 +1,243 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
zhuqi-lucas commented on code in PR #16395:
URL: https://github.com/apache/datafusion/pull/16395#discussion_r2148132009
##
datafusion-examples/examples/embedding_parquet_indexes.rs:
##
@@ -0,0 +1,243 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
pepijnve commented on issue #16396:
URL: https://github.com/apache/datafusion/issues/16396#issuecomment-2974057031
@alamb I have a first draft written up at
https://github.com/apache/datafusion-site/pull/75. I'm not a great blog writer,
so any help in getting this over the finish line would
pepijnve opened a new pull request, #75:
URL: https://github.com/apache/datafusion-site/pull/75
(no comment)
--
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,
pepijnve commented on PR #75:
URL: https://github.com/apache/datafusion-site/pull/75#issuecomment-2974061097
I've marked this as draft for now. I think I have the narrative arc I was
going for in place, but the text probably still needs some editing work.
--
This is an automated message f
comphead commented on PR #16409:
URL: https://github.com/apache/datafusion/pull/16409#issuecomment-2973561640
I'm personally intrigued tbh but I'd say the DF core should be agnostic of
specific data-driven architecture(like Spark) even if we do a lot of Spark
integration like Sail or Comet.
2010YOUY01 opened a new pull request, #16416:
URL: https://github.com/apache/datafusion/pull/16416
## Which issue does this PR close?
- Closes #.
## Rationale for this change
There are a large number of benchmarks now in `bench.sh`'s help message.
This PR gro
trompa commented on PR #1874:
URL:
https://github.com/apache/datafusion-comet/pull/1874#issuecomment-2973588238
code is now using a macro to generate the 3 hour, minute and second
functions.
new spark test to check literals are folded on jvm side
--
This is an automated message from t
pepijnve commented on PR #16398:
URL: https://github.com/apache/datafusion/pull/16398#issuecomment-2973548782
I've added a commit to this PR that:
- Makes the tests more robust. Rather than hanging when there's an issue
they will fail.
- Removes duplication from the tests
- Removes
shehabgamin commented on PR #16409:
URL: https://github.com/apache/datafusion/pull/16409#issuecomment-2973551244
> Got it @shehabgamin
>
> I'm seeing a lot of `slt` tests like
>
> ```
> #S
>
>
> #E
> #query
> #L
> ```
>
> which not very explanato
tglanz commented on issue #174:
URL:
https://github.com/apache/datafusion-comet/issues/174#issuecomment-2973618491
> Now that Comet supports DataFusion's `DataSourceExec` (when
`native_datafusion` scan is enabled) it should be much easier to support
`delta-rs`.
Is the suggestion mea
milenkovicm opened a new pull request, #1272:
URL: https://github.com/apache/datafusion-ballista/pull/1272
# Which issue does this PR close?
Closes #.
# Rationale for this change
pluggable `DistributionPolicy` provides a way to extend scheduler task
binding policies.
alamb merged PR #16416:
URL: https://github.com/apache/datafusion/pull/16416
--
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...@datafusi
alamb opened a new pull request, #16417:
URL: https://github.com/apache/datafusion/pull/16417
## Which issue does this PR close?
- Closes #.
## Rationale for this change
The [ASF process for inviting new PMC
members](https://www.apache.org/dev/pmc.html#pmcmembers
zhuqi-lucas commented on code in PR #16395:
URL: https://github.com/apache/datafusion/pull/16395#discussion_r2148133565
##
datafusion-examples/examples/embedding_parquet_indexes.rs:
##
@@ -0,0 +1,243 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
zhuqi-lucas commented on code in PR #16395:
URL: https://github.com/apache/datafusion/pull/16395#discussion_r214877
##
datafusion-examples/examples/embedding_parquet_indexes.rs:
##
@@ -0,0 +1,243 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
36 matches
Mail list logo
