iffyio merged PR #1924:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1924
--
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
kumarlokesh commented on PR #15776:
URL: https://github.com/apache/datafusion/pull/15776#issuecomment-3038251223
@alamb thanks for the review!
Updated the PR to address review comments.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
iffyio merged PR #1918:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1918
--
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
zhuqi-lucas commented on PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#issuecomment-3038122113
Thank you @alamb ! Addressed comments for the first round, but the image
still not add to the content due to it not showing well in my local.
--
This is an automated message fro
zhuqi-lucas commented on code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2186696545
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,251 @@
+## Extending Parquet with Embedded Indexes and Accelerating Query Processing
with Dat
zhuqi-lucas commented on code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2186669292
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,232 @@
+## Accelerating Query Processing in DataFusion with Embedded Parquet Indexes
+
+It’s a
solontsev commented on PR #1925:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1925#issuecomment-3038044031
I made it more permissive by allowing empty parentheses. I'd appreciate your
feedback on this
--
This is an automated message from the Apache Git Service.
To respond
adriangb commented on PR #16668:
URL: https://github.com/apache/datafusion/pull/16668#issuecomment-3038036678
note to self: check if these improvements trickle down into the physical
optimizers
--
This is an automated message from the Apache Git Service.
To respond to the message, please
solontsev opened a new pull request, #1925:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1925
Closes #1853
--
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.
T
adriangb commented on issue #14993:
URL: https://github.com/apache/datafusion/issues/14993#issuecomment-3038010228
> > Related blog from [@gatesn](https://github.com/gatesn)
> > https://blog.spiraldb.com/what-if-we-just-didnt-decompress-it/
>
> I now see how the vision of this can c
zhuqi-lucas commented on code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2186572247
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,232 @@
+## Accelerating Query Processing in DataFusion with Embedded Parquet Indexes
+
+It’s a
zhuqi-lucas commented on code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2186552221
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,232 @@
+## Accelerating Query Processing in DataFusion with Embedded Parquet Indexes
+
+It’s a
zhuqi-lucas commented on code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2186543546
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,232 @@
+## Accelerating Query Processing in DataFusion with Embedded Parquet Indexes
+
+It’s a
zhuqi-lucas commented on code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2186527671
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,232 @@
+## Accelerating Query Processing in DataFusion with Embedded Parquet Indexes
+
+It’s a
zhuqi-lucas commented on code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2186508081
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,232 @@
+## Accelerating Query Processing in DataFusion with Embedded Parquet Indexes
+
+It’s a
zhuqi-lucas commented on PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#issuecomment-3037856509
>
https://docs.google.com/presentation/d/1aFjTLEDJyDqzFZHgcmRxecCvLKKXV2OvyEpTQFCNZPw/edit?slide=id.g33d7337a5a0_0_85
Thank you @alamb for review and great suggestions! I wi
kosiew commented on PR #1177:
URL:
https://github.com/apache/datafusion-python/pull/1177#issuecomment-3037605845
You're welcome.
--
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
Standing-Man commented on code in PR #16675:
URL: https://github.com/apache/datafusion/pull/16675#discussion_r2186251681
##
datafusion/physical-plan/src/joins/sort_merge_join.rs:
##
@@ -2032,7 +2034,10 @@ impl SortMergeJoinStream {
let record_batch =
concat
fvj opened a new pull request, #16687:
URL: https://github.com/apache/datafusion/pull/16687
## Which issue does this PR close?
None. It's too small of a change for an issue, in my opinion. Happy to
retroactively create one, though.
## Rationale for this change
unfortunat
jkosh44 commented on code in PR #16608:
URL: https://github.com/apache/datafusion/pull/16608#discussion_r2186149622
##
datafusion/substrait/src/variation_const.rs:
##
@@ -53,6 +53,10 @@ pub const DATE_64_TYPE_VARIATION_REF: u32 = 1;
pub const DEFAULT_CONTAINER_TYPE_VARIATION_RE
Dandandan merged PR #16663:
URL: https://github.com/apache/datafusion/pull/16663
--
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...@data
Dandandan closed issue #16655: Update workspace to use Rust 1.88
URL: https://github.com/apache/datafusion/issues/16655
--
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 unsub
Dandandan commented on code in PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#discussion_r2186109216
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -319,19 +342,75 @@ impl TopK {
/// (a > 2 OR (a = 2 AND b < 3))
/// ```
fn update_filter(&mut s
Dandandan commented on code in PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#discussion_r2186107501
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -319,19 +342,75 @@ impl TopK {
/// (a > 2 OR (a = 2 AND b < 3))
/// ```
fn update_filter(&mut s
Dandandan commented on code in PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#discussion_r2186106170
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -319,19 +342,75 @@ impl TopK {
/// (a > 2 OR (a = 2 AND b < 3))
/// ```
fn update_filter(&mut s
liamzwbao commented on issue #16563:
URL: https://github.com/apache/datafusion/issues/16563#issuecomment-3037228593
I put up a fix in #16686. It works but not sure whether this is the best
place to apply the change.
--
This is an automated message from the Apache Git Service.
To respond t
liamzwbao commented on code in PR #16686:
URL: https://github.com/apache/datafusion/pull/16686#discussion_r2186078102
##
datafusion/core/tests/physical_optimizer/filter_pushdown/mod.rs:
##
@@ -289,7 +289,7 @@ fn test_no_pushdown_through_aggregates() {
Ok:
- F
Dandandan commented on PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#issuecomment-3037193167
I am taking a look now, see if I can find a thing
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
lliangyu-lin commented on code in PR #16633:
URL: https://github.com/apache/datafusion/pull/16633#discussion_r2186054590
##
datafusion/sqllogictest/bin/sqllogictests.rs:
##
@@ -243,7 +243,7 @@ async fn run_test_file_substrait_round_trip(
};
setup_scratch_dir(&relative_
yoavcloud opened a new pull request, #1924:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1924
We've encountered a problem parsing statements like `SELECT 1, sort FROM
tbl` which are valid in Snowflake.
The reason is that in the Snowflake dialect,
supports_projection_tr
liamzwbao opened a new pull request, #16686:
URL: https://github.com/apache/datafusion/pull/16686
## Which issue does this PR close?
- Closes #16563.
## Rationale for this change
Add the missing equivalence info so that optimizer can pick it up when
pruni
iffyio merged PR #1913:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1913
--
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
andygrove merged PR #1988:
URL: https://github.com/apache/datafusion-comet/pull/1988
--
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 code in PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#discussion_r2185866801
##
content/blog/datafusion-custom-parquet-index.md:
##
@@ -0,0 +1,232 @@
+## Accelerating Query Processing in DataFusion with Embedded Parquet Indexes
+
+It’s a commo
comphead commented on code in PR #16675:
URL: https://github.com/apache/datafusion/pull/16675#discussion_r2185865853
##
datafusion/physical-plan/src/joins/sort_merge_join.rs:
##
@@ -2032,7 +2034,10 @@ impl SortMergeJoinStream {
let record_batch =
concat_bat
comphead commented on code in PR #16675:
URL: https://github.com/apache/datafusion/pull/16675#discussion_r2185864775
##
datafusion/physical-plan/src/joins/sort_merge_join.rs:
##
@@ -2032,7 +2034,10 @@ impl SortMergeJoinStream {
let record_batch =
concat_bat
andygrove opened a new issue, #1989:
URL: https://github.com/apache/datafusion-comet/issues/1989
### What is the problem the feature request solves?
We currently only publish jars to Maven for Spark 3.4 and 3.5. We should now
include 4.0
See `dev/release/build-release-comet.sh`
andygrove opened a new pull request, #1988:
URL: https://github.com/apache/datafusion-comet/pull/1988
## Which issue does this PR close?
Closes #.
## Rationale for this change
## What changes are included in this PR?
## How are these changes
andygrove merged PR #78:
URL: https://github.com/apache/datafusion-site/pull/78
--
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...@dataf
iffyio merged PR #1892:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1892
--
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
rluvaton commented on PR #15700:
URL: https://github.com/apache/datafusion/pull/15700#issuecomment-3036807094
I would appreciate it, it would greatly help me
--
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
gabotechs commented on code in PR #16633:
URL: https://github.com/apache/datafusion/pull/16633#discussion_r2185709428
##
datafusion/sqllogictest/bin/sqllogictests.rs:
##
@@ -243,7 +243,7 @@ async fn run_test_file_substrait_round_trip(
};
setup_scratch_dir(&relative_pat
ding-young commented on PR #15700:
URL: https://github.com/apache/datafusion/pull/15700#issuecomment-3036770691
@rluvaton If you’d like, I can send a PR to your (fork's) branch that
resolve merge conflicts since I already have one. Anyway there were only minor
diffs to handle when I rebased
andygrove commented on code in PR #78:
URL: https://github.com/apache/datafusion-site/pull/78#discussion_r2185674170
##
content/blog/2025-07-01-datafusion-comet-0.9.0.md:
##
@@ -0,0 +1,176 @@
+---
+layout: post
+title: Apache DataFusion Comet 0.9.0 Release
+date: 2025-07-01
+aut
rluvaton commented on PR #15700:
URL: https://github.com/apache/datafusion/pull/15700#issuecomment-3036724762
So should I fix this PR conflicts? It seems like this pr has a change to be
merged
--
This is an automated message from the Apache Git Service.
To respond to the message, please
adriangb commented on code in PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#discussion_r2185669579
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -214,41 +238,39 @@ impl TopK {
let mut selected_rows = None;
-if let Some(filter) = self.f
geoffreyclaude opened a new pull request, #16685:
URL: https://github.com/apache/datafusion/pull/16685
## Which issue does this PR close?
- Draft attempt at #13583.
## How to review?
- The sqllogic tests in the
`datafusion/sqllogictest/test_files/match_recognize` cover e
ntjohnson1 opened a new pull request, #1179:
URL: https://github.com/apache/datafusion-python/pull/1179
# Which issue does this PR close?
Closes #1178
# Rationale for this change
Explained in issue. Trying to bring develop instructions up to current
experience getting started
ntjohnson1 opened a new issue, #1178:
URL: https://github.com/apache/datafusion-python/issues/1178
**Describe the bug**
Running the steps for how to develop on a clean mac leads to some missing
package errors.
1. Running maturin develop command initial errors that `protoc` can't be
fo
chenkovsky commented on code in PR #16342:
URL: https://github.com/apache/datafusion/pull/16342#discussion_r2185571139
##
datafusion/datasource/src/file_sink_config.rs:
##
@@ -77,13 +79,34 @@ pub trait FileSink: DataSink {
.runtime_env()
.object_store(&
debajyoti-truefoundry opened a new issue, #16684:
URL: https://github.com/apache/datafusion/issues/16684
### Describe the bug
I am on datafusion 47.
```rust
use arrow::array::Int64Array;
use arrow::datatypes::{DataType, Field, Schema};
use arrow::record_batch::RecordBat
ViggoC opened a new pull request, #16683:
URL: https://github.com/apache/datafusion/pull/16683
AggregateUDF is not used.
--
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
chenkovsky commented on code in PR #16342:
URL: https://github.com/apache/datafusion/pull/16342#discussion_r2185358781
##
datafusion/datasource/src/file_sink_config.rs:
##
@@ -77,13 +79,34 @@ pub trait FileSink: DataSink {
.runtime_env()
.object_store(&
Dandandan commented on code in PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#discussion_r2185350825
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -214,41 +238,39 @@ impl TopK {
let mut selected_rows = None;
-if let Some(filter) = self.
adriangb commented on code in PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#discussion_r2185318951
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -214,41 +238,39 @@ impl TopK {
let mut selected_rows = None;
-if let Some(filter) = self.f
Dandandan commented on code in PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#discussion_r2185314221
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -214,41 +238,39 @@ impl TopK {
let mut selected_rows = None;
-if let Some(filter) = self.
adriangb commented on PR #16433:
URL: https://github.com/apache/datafusion/pull/16433#issuecomment-3036131650
I did a bench run, confounding results:
```
┏━━┳━━┳━━┳━━━┓
┃ Query┃ main ┃ topk-filters ┃Change ┃
┡
brunal opened a new pull request, #16682:
URL: https://github.com/apache/datafusion/pull/16682
Reverts apache/datafusion#16342
After that change, one cannot write an empty RecordBatch with a schema to
parquet anymore.
Indeed, the logic added tries to write an empty recordbatch
brunal commented on PR #16342:
URL: https://github.com/apache/datafusion/pull/16342#issuecomment-3036100543
How can i send a rollback of this PR?
--
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
brunal commented on PR #16342:
URL: https://github.com/apache/datafusion/pull/16342#issuecomment-3036097695
User facing breakage: one cannot explicitly write an empty recordbatch that
has a schema anymore.
The tests in the PR don't have a schema so they don't reveal the issue.
--
T
brunal commented on code in PR #16342:
URL: https://github.com/apache/datafusion/pull/16342#discussion_r2185262228
##
datafusion/datasource/src/file_sink_config.rs:
##
@@ -77,13 +79,34 @@ pub trait FileSink: DataSink {
.runtime_env()
.object_store(&conf
timsaucer merged PR #1177:
URL: https://github.com/apache/datafusion-python/pull/1177
--
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...
timsaucer closed issue #1171: Add DOM-guarded CSS/JS injection to
DataFrameHtmlFormatter to prevent duplicate style/script inserts
URL: https://github.com/apache/datafusion-python/issues/1171
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
crepererum merged PR #16653:
URL: https://github.com/apache/datafusion/pull/16653
--
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...@dat
findepi commented on code in PR #16681:
URL: https://github.com/apache/datafusion/pull/16681#discussion_r2185166539
##
datafusion/expr/src/test/udf_equals.rs:
##
@@ -0,0 +1,187 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agre
adriangb commented on issue #16565:
URL: https://github.com/apache/datafusion/issues/16565#issuecomment-3035790931
Question: will we be able to handle something like `Dict(UInt32,
List(List(Struct(...`?
--
This is an automated message from the Apache Git Service.
To respond to the mes
kosiew opened a new pull request, #1177:
URL: https://github.com/apache/datafusion-python/pull/1177
## Which issue does this PR close?
- Closes #1171.
## Rationale for this change
This change simplifies the logic for injecting HTML styles in the
`DataFrameHtmlFormatter`
crepererum commented on PR #16653:
URL: https://github.com/apache/datafusion/pull/16653#issuecomment-3035482230
Rebased after #16672.
--
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
kosiew commented on issue #16565:
URL: https://github.com/apache/datafusion/issues/16565#issuecomment-3035480855
@adriangb,
> incorporating the struct-aware casting logic from #16371 into `CastExpr`
and `TryCastExpr`
Yes. I think it is a necessary step to explore the feasibilit
crepererum commented on PR #16653:
URL: https://github.com/apache/datafusion/pull/16653#issuecomment-3035427853
> Thanks @crepererum for taking care of that, I suspect it is motivated by
clippy complains
[rust-lang.github.io/rust-clippy/v0.0.212#large_enum_variant](https://rust-lang.github.
eliaperantoni commented on code in PR #1892:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1892#discussion_r2184969127
##
src/parser/mod.rs:
##
@@ -7866,7 +7866,7 @@ impl<'a> Parser<'a> {
}
pub(crate) fn parse_tag(&mut self) -> Result {
-let na
kosiew opened a new pull request, #16681:
URL: https://github.com/apache/datafusion/pull/16681
## Which issue does this PR close?
- Closes [#16677](https://github.com/apache/datafusion/issues/16677)
## Rationale for this change
The default implementation of `ScalarUDF
alamb merged PR #16679:
URL: https://github.com/apache/datafusion/pull/16679
--
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, #16680:
URL: https://github.com/apache/datafusion/pull/16680
## Which issue does this PR close?
- Closes https://github.com/apache/datafusion/issues/16626
## Rationale for this change
As @jonmmease pointed out, for people seeing t
zhuqi-lucas commented on PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#issuecomment-3035299849
Thank you @alamb , i will keep polishing it before you reviewing!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
alamb commented on PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#issuecomment-3035285253
This is amazing -- thank you @zhuqi-lucas and @2010YOUY01 -- I will review
this asap, but as today is a holiday in the US I may not have a chance to do so
until tomorrow.
--
This i
LiaCastaneda commented on code in PR #16445:
URL: https://github.com/apache/datafusion/pull/16445#discussion_r2184901077
##
datafusion/physical-plan/src/joins/hash_join.rs:
##
@@ -1039,12 +1196,50 @@ async fn collect_left_input(
let data = JoinLeftData::new(
hashma
alamb opened a new pull request, #16679:
URL: https://github.com/apache/datafusion/pull/16679
## Which issue does this PR close?
- Part of https://github.com/apache/datafusion/issues/16486
- Related to https://github.com/apache/datafusion/issues/16626
## Rationale for
zhuqi-lucas commented on PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#issuecomment-3035225130
> This post is great, I find the content easy to follow.
>
> I have a suggestion for the first paragraph though: perhaps we should
emphasize the motivation more clearly at t
alamb commented on issue #16486:
URL: https://github.com/apache/datafusion/issues/16486#issuecomment-3035205437
Ok, I have backported the three identified issues and I will now create a
release candidate
--
This is an automated message from the Apache Git Service.
To respond to the messag
alamb commented on issue #16677:
URL: https://github.com/apache/datafusion/issues/16677#issuecomment-3035199021
🤔 That is a tricky business
Maybe ScalarUDFImpl needs some way to have `eq` called on it too 🤔
--
This is an automated message from the Apache Git Service.
To respond to
alamb closed pull request #16673: Add a note about Boxing errors in upgrade
guide
URL: https://github.com/apache/datafusion/pull/16673
--
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 co
alamb commented on PR #16673:
URL: https://github.com/apache/datafusion/pull/16673#issuecomment-3035186129
@kosiew included this content in
- https://github.com/apache/datafusion/pull/16672
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
alamb commented on code in PR #16673:
URL: https://github.com/apache/datafusion/pull/16673#discussion_r2184855966
##
docs/source/library-user-guide/upgrading.md:
##
@@ -24,6 +24,38 @@
**Note:** DataFusion `49.0.0` has not been released yet. The information
provided in this sec
alamb merged PR #16657:
URL: https://github.com/apache/datafusion/pull/16657
--
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 commented on PR #16657:
URL: https://github.com/apache/datafusion/pull/16657#issuecomment-3035160095
FYI @timsaucer I am backporting this and making an 48.0.1 RC
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
alamb commented on PR #16659:
URL: https://github.com/apache/datafusion/pull/16659#issuecomment-3035158627
Thanks agian @dmitriibugakov and @blaginin
--
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
alamb merged PR #16659:
URL: https://github.com/apache/datafusion/pull/16659
--
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 merged PR #16630:
URL: https://github.com/apache/datafusion/pull/16630
--
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 closed issue #16629: Continue optimizing the CursorValues compare for
StringViewArray
URL: https://github.com/apache/datafusion/issues/16629
--
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
gabotechs commented on code in PR #16454:
URL: https://github.com/apache/datafusion/pull/16454#discussion_r2184812398
##
datafusion/core/src/physical_planner.rs:
##
@@ -1502,6 +1521,64 @@ fn get_null_physical_expr_pair(
Ok((Arc::new(null_value), physical_name))
}
+/// Qu
gabotechs commented on code in PR #16662:
URL: https://github.com/apache/datafusion/pull/16662#discussion_r2184807380
##
datafusion/proto/tests/cases/roundtrip_physical_plan.rs:
##
@@ -1736,3 +1737,55 @@ async fn roundtrip_physical_plan_node() {
let _ = plan.execute(0, ct
2010YOUY01 commented on PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#issuecomment-3035056285
This post is great, I find the content easy to follow.
I have a suggestion for the first paragraph though: perhaps we should
emphasize the motivation more clearly at the begi
Standing-Man commented on code in PR #16675:
URL: https://github.com/apache/datafusion/pull/16675#discussion_r2184734711
##
datafusion/physical-plan/src/joins/sort_merge_join.rs:
##
@@ -2032,7 +2034,10 @@ impl SortMergeJoinStream {
let record_batch =
concat
Standing-Man commented on code in PR #16674:
URL: https://github.com/apache/datafusion/pull/16674#discussion_r2184687258
##
datafusion/physical-plan/src/joins/symmetric_hash_join.rs:
##
@@ -1375,7 +1375,7 @@ impl SymmetricHashJoinStream {
}
Some
zhuqi-lucas commented on PR #79:
URL: https://github.com/apache/datafusion-site/pull/79#issuecomment-3034853896
I am not expert for blog, welcome folks to polish it together, thanks a lot!
cc @alamb
--
This is an automated message from the Apache Git Service.
To respond to the message, p
zhuqi-lucas commented on issue #16372:
URL: https://github.com/apache/datafusion/issues/16372#issuecomment-3034853243
Submit the draft blog:
https://github.com/apache/datafusion-site/pull/79
I am not expert for blog, welcome folks to polish it together, thanks a lot!
--
This is a
zhuqi-lucas opened a new pull request, #79:
URL: https://github.com/apache/datafusion-site/pull/79
Try to blog our work for the custom parquet example for datafusion:
https://github.com/apache/datafusion/pull/16395
And also close this ticket:
https://github.com/apache/dat
2010YOUY01 commented on code in PR #16675:
URL: https://github.com/apache/datafusion/pull/16675#discussion_r2184636583
##
datafusion/physical-plan/src/joins/sort_merge_join.rs:
##
@@ -2032,7 +2034,10 @@ impl SortMergeJoinStream {
let record_batch =
concat_b
2010YOUY01 commented on code in PR #16674:
URL: https://github.com/apache/datafusion/pull/16674#discussion_r2184635238
##
datafusion/physical-plan/src/joins/symmetric_hash_join.rs:
##
@@ -1375,7 +1375,7 @@ impl SymmetricHashJoinStream {
}
Some((
1 - 100 of 111 matches
Mail list logo
