westhide commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2028175201
##
ballista/scheduler/src/scheduler_server/grpc.rs:
##
@@ -125,10 +125,15 @@ impl SchedulerGrpc
let mut tasks = vec![];
for (_
westhide commented on PR #1216:
URL:
https://github.com/apache/datafusion-ballista/pull/1216#issuecomment-2777671990
> > Q1: As the `BallistaFlightService` keep listenning on each
Executor,writting it allow client to send a `do_get` request, and without check
`FetchPartition` action's `pat
adriangb commented on code in PR #15561:
URL: https://github.com/apache/datafusion/pull/15561#discussion_r2027781065
##
datafusion/core/src/datasource/physical_plan/parquet.rs:
##
@@ -1445,119 +1465,26 @@ mod tests {
// batch1: c1(string)
let batch1 = string_ba
adriangb commented on code in PR #15561:
URL: https://github.com/apache/datafusion/pull/15561#discussion_r2027668758
##
datafusion/core/src/datasource/physical_plan/parquet.rs:
##
@@ -1445,119 +1465,26 @@ mod tests {
// batch1: c1(string)
let batch1 = string_ba
Adez017 commented on issue #6781:
URL: https://github.com/apache/datafusion/issues/6781#issuecomment-2776464521
hi @alamb , it sounds pretty interesting ! .
--
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
andygrove closed issue #1235: Optimize repartitioning logic in
ShuffleWriterExec using interleave_record_batch
URL: https://github.com/apache/datafusion-comet/issues/1235
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
mbutrovich commented on PR #15537:
URL: https://github.com/apache/datafusion/pull/15537#issuecomment-2776452713
https://github.com/apache/parquet-testing/pull/73 merged so I updated the
parquet-testing dependency. Now waiting on an arrow-rs release and DF bumping
to that version.
--
This
mbutrovich commented on issue #1228:
URL:
https://github.com/apache/datafusion-comet/issues/1228#issuecomment-2776728769
> Has there been an update on this?
>
> My workflow specifies a schema to my read function through a JSON file,
date-related fields are specified to be timestamp t
qstommyshu commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027702690
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -4665,18 +4674,18 @@ Projection: person.id, person.age
}
#[test]
-fn test_prepare_statement_infer_types_fr
Adez017 commented on PR #1:
URL: https://github.com/apache/datafusion/pull/1#issuecomment-2777473262
hey @alamb , i'm pretty confused over here . could you please help to
specify what is failing for ?
--
This is an automated message from the Apache Git Service.
To respond to t
xudong963 commented on issue #15554:
URL: https://github.com/apache/datafusion/issues/15554#issuecomment-2777425499
dup with https://github.com/apache/datafusion/issues/15571
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
alamb closed issue #15530: Internal error: PhysicalExpr Column references bound
error, Failure in spilling for `AggregateMode::Single`
URL: https://github.com/apache/datafusion/issues/15530
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
alamb commented on issue #13037:
URL: https://github.com/apache/datafusion/issues/13037#issuecomment-2776834231
Thanks @ByteBaker -- I don't have any strong opinion / advice here
unfortunately
--
This is an automated message from the Apache Git Service.
To respond to the message, please
Adez017 commented on issue #6781:
URL: https://github.com/apache/datafusion/issues/6781#issuecomment-2776464810
take
--
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 unsu
xudong963 closed issue #15554: Audit-check fails in main branch
URL: https://github.com/apache/datafusion/issues/15554
--
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 unsubs
jayzhan211 commented on issue #15511:
URL: https://github.com/apache/datafusion/issues/15511#issuecomment-2777328776
> You are also right here about the cost, but what if we can have 2 modes
for benchmarks, one for the actual benchmarking purpose, and one with just to
validate. If it is in
alamb closed issue #15358: limit max disk usage for spilling queries
URL: https://github.com/apache/datafusion/issues/15358
--
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
qstommyshu commented on issue #14692:
URL: https://github.com/apache/datafusion/issues/14692#issuecomment-2777376142
take
--
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
qstommyshu commented on issue #14692:
URL: https://github.com/apache/datafusion/issues/14692#issuecomment-2777376035
This issue looks interesting to me. I'll try to work on it!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
chenkovsky commented on issue #15565:
URL: https://github.com/apache/datafusion/issues/15565#issuecomment-2777362621
take
--
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 opened a new pull request, #15574:
URL: https://github.com/apache/datafusion/pull/15574
## Which issue does this PR close?
- Closes #15509 .
## Rationale for this change
## What changes are included in this PR?
remove ::INT::DATE in clickbench.
chenkovsky commented on issue #15569:
URL: https://github.com/apache/datafusion/issues/15569#issuecomment-2777357348
take
--
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
jayzhan211 commented on issue #15571:
URL: https://github.com/apache/datafusion/issues/15571#issuecomment-2777340299
Can we try `proc-macro-error2` or the latest version of `proc-macro-error`?
If it is not trivial or causing too much breaking change, then we can add a
warning instead
--
jayzhan211 merged PR #15281:
URL: https://github.com/apache/datafusion/pull/15281
--
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
alamb commented on PR #15566:
URL: https://github.com/apache/datafusion/pull/15566#issuecomment-2776871923
This is first thing on my list to review 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
comphead closed issue #1228: [comet-parquet-exec] Track remaining test failures
in POC 1 & 2
URL: https://github.com/apache/datafusion-comet/issues/1228
--
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
jayzhan211 commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2777259273
Thanks @suibianwanwank
--
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 specifi
jayzhan211 closed issue #15032: Similar to the "count-bug" case that produces
incorrect results
URL: https://github.com/apache/datafusion/issues/15032
--
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
jayzhan211 commented on PR #15564:
URL: https://github.com/apache/datafusion/pull/15564#issuecomment-2777258566
Thanks @xudong963 @alamb
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the speci
wiedld commented on PR #15409:
URL: https://github.com/apache/datafusion/pull/15409#issuecomment-2777243569
Thanks for the review.
Haven't had time to do the updates.
Converting to draft, and will mark ready again after updates.
--
This is an automated message from the Apache Git
Lordworms commented on issue #14958:
URL: https://github.com/apache/datafusion/issues/14958#issuecomment-2777223899
Like what duckdb did
## 1. Detect Correlated Columns
- Walk subquery expressions and collect all outer (correlated) column
references.
- Represent them as `Expr::Outer
adriangb commented on issue #15387:
URL: https://github.com/apache/datafusion/issues/15387#issuecomment-2777214161
But is that possible to evaluate via a simplifier? I'd think that in general
we don't know that until execution time.
--
This is an automated message from the Apache
alamb commented on code in PR #15462:
URL: https://github.com/apache/datafusion/pull/15462#discussion_r2026912500
##
datafusion/physical-expr/src/expressions/binary.rs:
##
@@ -805,6 +811,47 @@ impl BinaryExpr {
}
}
+/// Check if it meets the short-circuit condition
+///
Jiashu-Hu commented on issue #15571:
URL: https://github.com/apache/datafusion/issues/15571#issuecomment-2777171963
For now, it might be best to just allow this warning. Since everything is
working fine as it is. Alternatively we could try to use 'proc-macro-error2'
instead, but that might
Jiashu-Hu commented on issue #15571:
URL: https://github.com/apache/datafusion/issues/15571#issuecomment-2777150559
take
--
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
adriangb commented on code in PR #15561:
URL: https://github.com/apache/datafusion/pull/15561#discussion_r2027851655
##
datafusion/datasource-parquet/src/opener.rs:
##
@@ -295,3 +312,89 @@ fn create_initial_plan(
// default to scanning all row groups
Ok(ParquetAccessPl
andygrove commented on issue #1010:
URL:
https://github.com/apache/datafusion-comet/issues/1010#issuecomment-2775877457
Closing as duplicate of https://github.com/apache/datafusion-comet/issues/947
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
qstommyshu commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027712552
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -3388,26 +3389,15 @@ fn
ident_normalization_parser_options_ident_normalization() -> ParserOptions {
}
}
andygrove closed issue #1450: 【TPCH】Comet do not show performance advantages
over native Spark？
URL: https://github.com/apache/datafusion-comet/issues/1450
--
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
andygrove commented on issue #1157:
URL:
https://github.com/apache/datafusion-comet/issues/1157#issuecomment-2775885159
Closing this issue since it is for an issue with a Databricks repo that we
do not control
--
This is an automated message from the Apache Git Service.
To respond to the
andygrove closed issue #230: Add more developer documentation
URL: https://github.com/apache/datafusion-comet/issues/230
--
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 unsu
andygrove commented on issue #1450:
URL:
https://github.com/apache/datafusion-comet/issues/1450#issuecomment-2775796160
I'll close this issue for now since I cannot reproduce. Please feel free to
reopen it if you'd like to continue with this.
--
This is an automated message from the Apac
iffyio opened a new pull request, #1796:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1796
Fixes [lint failures in
CI](https://github.com/apache/datafusion-sqlparser-rs/actions/runs/14236730212/job/39938621832?pr=1790)
from the latest rust release
--
This is an automated m
alamb commented on PR #1:
URL: https://github.com/apache/datafusion/pull/1#issuecomment-2776823505
It looks like there are a few small errors to fix to get a clean CI run
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
adriangb commented on code in PR #15561:
URL: https://github.com/apache/datafusion/pull/15561#discussion_r2027772249
##
datafusion/sqllogictest/test_files/parquet.slt:
##
@@ -625,7 +625,7 @@ physical_plan
01)CoalesceBatchesExec: target_batch_size=8192
02)--FilterExec: column1@
alamb commented on code in PR #15561:
URL: https://github.com/apache/datafusion/pull/15561#discussion_r2027656995
##
datafusion/core/src/datasource/physical_plan/parquet.rs:
##
@@ -1445,119 +1465,26 @@ mod tests {
// batch1: c1(string)
let batch1 = string_batch
alamb commented on PR #12302:
URL: https://github.com/apache/datafusion/pull/12302#issuecomment-2776791139
FYI @rluvaton and I have noticed a `clone()` introduced in this PR appearing
in some traces:
- https://github.com/apache/datafusion/issues/15573
--
This is an automated messag
qstommyshu commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027712552
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -3388,26 +3389,15 @@ fn
ident_normalization_parser_options_ident_normalization() -> ParserOptions {
}
}
qstommyshu commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027712552
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -3388,26 +3389,15 @@ fn
ident_normalization_parser_options_ident_normalization() -> ParserOptions {
}
}
qstommyshu commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027712552
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -3388,26 +3389,15 @@ fn
ident_normalization_parser_options_ident_normalization() -> ParserOptions {
}
}
alamb commented on issue #15271:
URL: https://github.com/apache/datafusion/issues/15271#issuecomment-2776837729
> so If I have a lot of spill files or if every batch is really huge
(contains very large lists - like result for array_agg on large dataset) we
have all of this in memory.
comphead commented on issue #1228:
URL:
https://github.com/apache/datafusion-comet/issues/1228#issuecomment-2776744809
Closing this in favor of #1441
--
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
mkgada commented on issue #1228:
URL:
https://github.com/apache/datafusion-comet/issues/1228#issuecomment-2776590893
Has there been an update on this?
My workflow specifies a schema to my read function through a JSON file,
date-related fields are specified to be timestamp type.
alamb commented on PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#issuecomment-2776729981
is this the last one?
--
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 comme
qstommyshu commented on PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#issuecomment-2776882350
> is this the last one?
This is the last one for migrating tests in `tests/sql_integration.rs`.
There are still some cases in `tests/cases/plan_to_sql.rs` and
`tests
rluvaton commented on PR #15562:
URL: https://github.com/apache/datafusion/pull/15562#issuecomment-2776872261
I removed the custom fixed size queue and replaced with matching vec deque
functions that are 0-cost as it's just index manipulation like my prev impl
--
This is an automated m
qstommyshu commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027712552
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -3388,26 +3389,15 @@ fn
ident_normalization_parser_options_ident_normalization() -> ParserOptions {
}
}
rluvaton commented on code in PR #15562:
URL: https://github.com/apache/datafusion/pull/15562#discussion_r2027711140
##
datafusion/physical-plan/src/sorts/merge.rs:
##
@@ -241,10 +239,13 @@ impl SortPreservingMergeStream {
_ => {
//
alamb opened a new issue, #15571:
URL: https://github.com/apache/datafusion/issues/15571
### Describe the bug
We are seeing a cargo audit failure on @zebsme 's PR:
https://github.com/apache/datafusion/pull/15454
```
Crate: proc-macro-error
Version: 1.0.4
Warn
andygrove closed issue #1599: Remove unwraps in `hash_array_small_decimal`
URL: https://github.com/apache/datafusion-comet/issues/1599
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
qstommyshu commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027702690
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -4665,18 +4674,18 @@ Projection: person.id, person.age
}
#[test]
-fn test_prepare_statement_infer_types_fr
alamb commented on PR #15541:
URL: https://github.com/apache/datafusion/pull/15541#issuecomment-2776844631
Thanks @oznur-synnada and @berkaysynnada
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go t
alamb commented on issue #15069:
URL: https://github.com/apache/datafusion/issues/15069#issuecomment-2776844057
> So I don't believe automatically running the full sqllogictests through
Substrait is feasible today. A more realistic goal would be to have a few
hand-crafted queries (TPCH quer
alamb merged PR #15547:
URL: https://github.com/apache/datafusion/pull/15547
--
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 #15547:
URL: https://github.com/apache/datafusion/pull/15547#issuecomment-2776824435
Thanks again @kumarlokesh
--
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
alamb closed issue #15546: Add `topk` information into `tree` explain plans
URL: https://github.com/apache/datafusion/issues/15546
--
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
alamb commented on issue #6781:
URL: https://github.com/apache/datafusion/issues/6781#issuecomment-2776822081
Thanks @Adez017 --
You can make a blog post for the DataFusion blog on by making a PR to this
repo: https://github.com/alamb/datafusion-site
--
This is an automated messa
alamb commented on issue #15323:
URL: https://github.com/apache/datafusion/issues/15323#issuecomment-2776820608
> I think I have the the same problem but in `AggregateExec` when using
`row_hash`, as it spills as well and use `SortPreservingMergeStream`.
>
> I think the solution should
jazracherif commented on code in PR #84:
URL: https://github.com/apache/datafusion-ray/pull/84#discussion_r2027648678
##
tpch/tpcbench.py:
##
@@ -186,8 +186,28 @@ def main(
args = parser.parse_args()
+if (args.qnum != -1 and args.query is not None):
+print("
alamb commented on PR #15564:
URL: https://github.com/apache/datafusion/pull/15564#issuecomment-2776794019
Thank you @xudong963
--
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 comme
alamb commented on code in PR #12302:
URL: https://github.com/apache/datafusion/pull/12302#discussion_r2027651936
##
datafusion/physical-plan/src/sorts/merge.rs:
##
@@ -154,14 +165,36 @@ impl SortPreservingMergeStream {
if self.aborted {
return Poll::Ready(
blaginin commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027650462
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -3388,26 +3389,15 @@ fn
ident_normalization_parser_options_ident_normalization() -> ParserOptions {
}
}
alamb commented on PR #15562:
URL: https://github.com/apache/datafusion/pull/15562#issuecomment-2776787483
BTW I filed a ticket to track this as I have seen it too
- https://github.com/apache/datafusion/issues/15573
--
This is an automated message from the Apache Git Service.
To respon
alamb opened a new issue, #15573:
URL: https://github.com/apache/datafusion/issues/15573
### Is your feature request related to a problem or challenge?
Both @rluvaton and I have seen
https://github.com/user-attachments/assets/cd91c702-51fa-45b7-9214-7913c1281161";
/>
!
dennyglee commented on issue #174:
URL:
https://github.com/apache/datafusion-comet/issues/174#issuecomment-2776310481
Oh brilliant! 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
alamb commented on code in PR #15562:
URL: https://github.com/apache/datafusion/pull/15562#discussion_r2027641047
##
datafusion/physical-plan/src/sorts/merge.rs:
##
@@ -241,10 +239,13 @@ impl SortPreservingMergeStream {
_ => {
// If
milenkovicm commented on PR #1224:
URL:
https://github.com/apache/datafusion-ballista/pull/1224#issuecomment-2776764135
clippy errors should be fixed in #1225
--
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
geoffreyclaude opened a new issue, #15559:
URL: https://github.com/apache/datafusion/issues/15559
### Is your feature request related to a problem or challenge?
Currently, the benchmarks folder in DataFusion does not include dedicated
benchmarks for TopK queries (i.e., queries formatt
adriangb commented on PR #15301:
URL: https://github.com/apache/datafusion/pull/15301#issuecomment-2776668146
I'm closing this now massive PR in favor of splitting it up into units of
work https://github.com/apache/datafusion/issues/15512#issuecomment-2776631138
Thank you all for the
alamb commented on code in PR #15567:
URL: https://github.com/apache/datafusion/pull/15567#discussion_r2027613057
##
datafusion/sql/tests/sql_integration.rs:
##
@@ -15,6 +15,7 @@
// specific language governing permissions and limitations
// under the License.
+use core::pani
alamb commented on code in PR #15547:
URL: https://github.com/apache/datafusion/pull/15547#discussion_r2027592484
##
datafusion/physical-plan/src/sorts/sort.rs:
##
@@ -1112,7 +1112,10 @@ impl DisplayAs for SortExec {
impl ExecutionPlan for SortExec {
fn name(&self) -> &'
alamb commented on PR #15531:
URL: https://github.com/apache/datafusion/pull/15531#issuecomment-2776696614
Thank you @rluvaton
--
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 commen
alamb closed issue #15530: Internal error: PhysicalExpr Column references bound
error, Failure in spilling for `AggregateMode::Single`
URL: https://github.com/apache/datafusion/issues/15530
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
alamb merged PR #15531:
URL: https://github.com/apache/datafusion/pull/15531
--
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 #15520:
URL: https://github.com/apache/datafusion/pull/15520#issuecomment-2776693763
Thanks again @2010YOUY01
--
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 c
alamb merged PR #15520:
URL: https://github.com/apache/datafusion/pull/15520
--
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
parthchandra commented on issue #692:
URL:
https://github.com/apache/datafusion-comet/issues/692#issuecomment-2776495289
Seems like a perennial issue. This signature changes in every release it
appears (it is private after all).
https://github.com/apache/datafusion-comet/issues/1576
--
adriangb commented on issue #15037:
URL: https://github.com/apache/datafusion/issues/15037#issuecomment-2776667178
Sorry closed the issue instead of the PR, my bad!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
adriangb closed issue #15037: Dynamic pruning filters from TopK state (optimize
`ORDER BY LIMIT` queries)
URL: https://github.com/apache/datafusion/issues/15037
--
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
adriangb opened a new pull request, #15568:
URL: https://github.com/apache/datafusion/pull/15568
Work towards #15512
--
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 unsu
milenkovicm opened a new pull request, #1224:
URL: https://github.com/apache/datafusion-ballista/pull/1224
# Which issue does this PR close?
Closes #1223 .
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing changes?
rluvaton commented on issue #15323:
URL: https://github.com/apache/datafusion/issues/15323#issuecomment-2776605858
I think I have the the same problem but in `AggregateExec` when using
`row_hash`, as it spills as well and use `SortPreservingMergeStream`.
I think the solution should ac
milenkovicm opened a new pull request, #1225:
URL: https://github.com/apache/datafusion-ballista/pull/1225
# Which issue does this PR close?
Closes #.
# Rationale for this change
# What changes are included in this PR?
- fix clippy issues after rust 1.86 u
adriangb commented on code in PR #15561:
URL: https://github.com/apache/datafusion/pull/15561#discussion_r2027429964
##
datafusion/core/src/datasource/file_format/parquet.rs:
##
@@ -67,13 +67,13 @@ pub(crate) mod test_util {
.into_iter()
.zip(tmp_files.
LiaCastaneda commented on code in PR #15438:
URL: https://github.com/apache/datafusion/pull/15438#discussion_r2025709682
##
datafusion/physical-expr/src/equivalence/projection.rs:
##
@@ -66,9 +66,9 @@ impl ProjectionMapping {
let idx = col.index();
LiaCastaneda commented on code in PR #15438:
URL: https://github.com/apache/datafusion/pull/15438#discussion_r2025709682
##
datafusion/physical-expr/src/equivalence/projection.rs:
##
@@ -66,9 +66,9 @@ impl ProjectionMapping {
let idx = col.index();
Dandandan commented on code in PR #15544:
URL: https://github.com/apache/datafusion/pull/15544#discussion_r2027495310
##
datafusion/functions-aggregate/src/array_agg.rs:
##
@@ -500,11 +548,30 @@ impl Accumulator for OrderSensitiveArrayAggAccumulator {
return Ok(());
andygrove closed issue #1553: Native scan panic with native_iceberg_compat on
hdfs
URL: https://github.com/apache/datafusion-comet/issues/1553
--
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 spe
andygrove merged PR #1604:
URL: https://github.com/apache/datafusion-comet/pull/1604
--
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...@
adriangb opened a new pull request, #15566:
URL: https://github.com/apache/datafusion/pull/15566
Work towards #15037
--
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
1 - 100 of 187 matches
Mail list logo
