Dandandan commented on code in PR #13581:
URL: https://github.com/apache/datafusion/pull/13581#discussion_r1863046233
##
datafusion/functions-aggregate/src/correlation.rs:
##
@@ -263,3 +283,307 @@ impl Accumulator for CorrelationAccumulator {
Ok(())
}
}
+
+#[deriv
Dandandan commented on code in PR #13581:
URL: https://github.com/apache/datafusion/pull/13581#discussion_r1863046233
##
datafusion/functions-aggregate/src/correlation.rs:
##
@@ -263,3 +283,307 @@ impl Accumulator for CorrelationAccumulator {
Ok(())
}
}
+
+#[deriv
Dandandan merged PR #13572:
URL: https://github.com/apache/datafusion/pull/13572
--
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 #13573: Test TPCH with sort merge join
URL: https://github.com/apache/datafusion/issues/13573
--
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 unsubscr
Dandandan commented on PR #13572:
URL: https://github.com/apache/datafusion/pull/13572#issuecomment-2507250856
Thanks for the review @comphead
--
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
berkaysynnada commented on code in PR #13540:
URL: https://github.com/apache/datafusion/pull/13540#discussion_r1863073808
##
datafusion/functions-table/src/generate_series.rs:
##
@@ -0,0 +1,180 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contrib
sgrebnov opened a new pull request, #13599:
URL: https://github.com/apache/datafusion/pull/13599
## Which issue does this PR close?
Follow-up item for [Support unparsing plans after applying the
`optimize_projections` rule
PR](https://github.com/apache/datafusion/pull/13267), focusin
sgrebnov commented on code in PR #13599:
URL: https://github.com/apache/datafusion/pull/13599#discussion_r1863025876
##
datafusion/sql/tests/cases/plan_to_sql.rs:
##
@@ -926,12 +926,25 @@ fn test_table_scan_pushdown() -> Result<()> {
let query_from_table_scan_with_projectio
sgrebnov commented on code in PR #13599:
URL: https://github.com/apache/datafusion/pull/13599#discussion_r1863025876
##
datafusion/sql/tests/cases/plan_to_sql.rs:
##
@@ -926,12 +926,25 @@ fn test_table_scan_pushdown() -> Result<()> {
let query_from_table_scan_with_projectio
findepi commented on code in PR #13580:
URL: https://github.com/apache/datafusion/pull/13580#discussion_r1861731143
##
datafusion/common/src/cast.rs:
##
@@ -93,14 +131,9 @@ pub fn as_string_view_array(array: &dyn Array) ->
Result<&StringViewArray> {
Ok(downcast_value!(arra
iffyio commented on PR #1552:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1552#issuecomment-2505619015
Ah yeah using a dialect method for this makes sense in that case
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
adi-kmt closed pull request #979: chore: Fix jdk documentation for spark
URL: https://github.com/apache/datafusion-comet/pull/979
--
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.
peter-toth commented on code in PR #13589:
URL: https://github.com/apache/datafusion/pull/13589#discussion_r1862019495
##
datafusion/expr/src/logical_plan/tree_node.rs:
##
@@ -710,13 +714,12 @@ impl LogicalPlan {
node: &LogicalPlan,
f: &mut F,
peter-toth commented on code in PR #13589:
URL: https://github.com/apache/datafusion/pull/13589#discussion_r1862019495
##
datafusion/expr/src/logical_plan/tree_node.rs:
##
@@ -710,13 +714,12 @@ impl LogicalPlan {
node: &LogicalPlan,
f: &mut F,
peter-toth commented on PR #13589:
URL: https://github.com/apache/datafusion/pull/13589#issuecomment-2505915082
> I verified test coverage by running the test without the code changes and
it failed
>
> ```
> assertion failed: !filter_found
> thread 'logical_plan::plan::tests::te
alamb commented on code in PR #1556:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1556#discussion_r1862005838
##
src/ast/mod.rs:
##
@@ -7653,6 +7653,7 @@ impl fmt::Display for ShowStatementInParentType {
pub struct ShowStatementIn {
pub clause: ShowStatemen
2010YOUY01 commented on code in PR #13581:
URL: https://github.com/apache/datafusion/pull/13581#discussion_r1862055020
##
datafusion/functions-aggregate-common/src/aggregate/groups_accumulator/accumulate.rs:
##
@@ -371,6 +371,87 @@ pub fn accumulate(
}
}
+/// Accumulates
notfilippo opened a new issue, #13593:
URL: https://github.com/apache/datafusion/issues/13593
### Is your feature request related to a problem or challenge?
The `datafusion-substrait` package currently depends on the `datafusion`
crate with all default features **enabled**. This can f
notfilippo commented on issue #13593:
URL: https://github.com/apache/datafusion/issues/13593#issuecomment-2505968846
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
2010YOUY01 commented on issue #13548:
URL: https://github.com/apache/datafusion/issues/13548#issuecomment-2505971804
> I am using a M3 Macbook with 16 GB of RAM. How much RAM does your machine
have? Perhaps DataFusion only struggles with query 9 when the machine doesn't
have lots of extra R
notfilippo opened a new pull request, #13594:
URL: https://github.com/apache/datafusion/pull/13594
## Which issue does this PR close?
Closes #13593
## What changes are included in this PR?
- `physical` feature, which enables production and consumption of physical
substr
Weijun-H merged PR #13580:
URL: https://github.com/apache/datafusion/pull/13580
--
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
Weijun-H commented on PR #13580:
URL: https://github.com/apache/datafusion/pull/13580#issuecomment-2505646747
Thanks @alamb for reviewing
--
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
Weijun-H merged PR #13587:
URL: https://github.com/apache/datafusion/pull/13587
--
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 commented on code in PR #1541:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1541#discussion_r1861776679
##
src/parser/mod.rs:
##
@@ -2935,12 +2935,23 @@ impl<'a> Parser<'a> {
})
} else if Token::LBracket == tok {
if diale
peter-toth opened a new pull request, #13589:
URL: https://github.com/apache/datafusion/pull/13589
## Which issue does this PR close?
Part of https://github.com/apache/datafusion/issues/8913.
## Rationale for this change
`LogicalPlan::..._with_subqueries()` methods don't
peter-toth commented on PR #13046:
URL: https://github.com/apache/datafusion/pull/13046#issuecomment-2505602715
@andygrove , I've rebased my PR on latest `main` if you want to give it a
try.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
peter-toth commented on PR #13589:
URL: https://github.com/apache/datafusion/pull/13589#issuecomment-2505714389
cc @alamb , @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 to the spe
Dandandan commented on code in PR #13581:
URL: https://github.com/apache/datafusion/pull/13581#discussion_r1861872899
##
datafusion/functions-aggregate-common/src/aggregate/groups_accumulator/accumulate.rs:
##
@@ -371,6 +371,87 @@ pub fn accumulate(
}
}
+/// Accumulates
Dandandan commented on code in PR #13581:
URL: https://github.com/apache/datafusion/pull/13581#discussion_r1861872899
##
datafusion/functions-aggregate-common/src/aggregate/groups_accumulator/accumulate.rs:
##
@@ -371,6 +371,87 @@ pub fn accumulate(
}
}
+/// Accumulates
leoyvens opened a new pull request, #13590:
URL: https://github.com/apache/datafusion/pull/13590
## Which issue does this PR close?
My motivation was to improve DF testing of float outputs, even at the
least-significant digits.
The situation in #13569 seemed a bit uncomfortable
Dandandan merged PR #13579:
URL: https://github.com/apache/datafusion/pull/13579
--
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 opened a new issue, #13591:
URL: https://github.com/apache/datafusion/issues/13591
### Is your feature request related to a problem or challenge?
As noted by @comphead in
https://github.com/apache/datafusion/pull/13579#pullrequestreview-2465606280
We can move the repeate
findepi opened a new pull request, #13592:
URL: https://github.com/apache/datafusion/pull/13592
This is helpful for simple function implementations or function stubs, when
`Signature::exact` is not desired.
--
This is an automated message from the Apache Git Service.
To respond to the
alamb commented on PR #13590:
URL: https://github.com/apache/datafusion/pull/13590#issuecomment-2505853129
Thank you @leoyvens -- this looks epic. I will review this PR but I may not
have a chance to do so for a day or two. It looks awesome
--
This is an automated message from the Apache
alamb commented on code in PR #13589:
URL: https://github.com/apache/datafusion/pull/13589#discussion_r1861976956
##
datafusion/expr/src/logical_plan/tree_node.rs:
##
@@ -710,13 +714,12 @@ impl LogicalPlan {
node: &LogicalPlan,
f: &mut F,
) ->
alamb commented on code in PR #13585:
URL: https://github.com/apache/datafusion/pull/13585#discussion_r1861280467
##
datafusion/physical-plan/src/aggregates/topk/heap.rs:
##
@@ -151,10 +151,11 @@ where
}
fn drain(&mut self) -> (ArrayRef, Vec) {
+let nulls = N
Dandandan commented on code in PR #13581:
URL: https://github.com/apache/datafusion/pull/13581#discussion_r1862087935
##
datafusion/functions-aggregate-common/src/aggregate/groups_accumulator/accumulate.rs:
##
@@ -371,6 +371,69 @@ pub fn accumulate(
}
}
+/// Accumulates
milenkovicm closed pull request #1141: chore: Update python files
URL: https://github.com/apache/datafusion-ballista/pull/1141
--
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
jayzhan211 commented on PR #13404:
URL: https://github.com/apache/datafusion/pull/13404#issuecomment-2506894562
@Omega359
How about we make this configurable?
Enable implicit coercion if we want the ease of use and the casting cost is
acceptable, disable it if we want prefer explicit
jayzhan211 commented on PR #13404:
URL: https://github.com/apache/datafusion/pull/13404#issuecomment-2506897643
And set implicit coercion as the default
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
jonathanc-n commented on PR #13404:
URL: https://github.com/apache/datafusion/pull/13404#issuecomment-2506910402
@jayzhan211 That sounds good. However, I think implicit coercion should be
the default or it'll cause regressions for users. Are you able to open this pr
back up, i can add the c
jayzhan211 commented on PR #13404:
URL: https://github.com/apache/datafusion/pull/13404#issuecomment-2506916421
An alternative approach is that we need to differentiate `string literal`
and `varchar` like Postgres an DuckDB. Only untyped `string literal` is able to
cast to any other types,
Omega359 commented on PR #13527:
URL: https://github.com/apache/datafusion/pull/13527#issuecomment-2506774637
> > Maybe we could change the semantics so that `SessionConfig` has a
`Arc` which was cloned when it was modified
(`Arc::unwrap_or_clone()` style) 🤔
>
> Certainly possible, I
comphead commented on PR #13595:
URL: https://github.com/apache/datafusion/pull/13595#issuecomment-2506830432
Since this is a first time contribution I'll be waiting for another review
before merging it in
--
This is an automated message from the Apache Git Service.
To respond to the mess
jonahgao merged PR #13596:
URL: https://github.com/apache/datafusion/pull/13596
--
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
jonahgao closed issue #13597: Fix build issues on latest stable Rust toolchain
(1.83)
URL: https://github.com/apache/datafusion/issues/13597
--
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
jonahgao commented on PR #13596:
URL: https://github.com/apache/datafusion/pull/13596#issuecomment-2506949744
Thanks @findepi @comphead @Dandandan
--
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
github-actions[bot] closed pull request #12524: Enable parquet pushdown_filter
by default
URL: https://github.com/apache/datafusion/pull/12524
--
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
github-actions[bot] commented on PR #12266:
URL: https://github.com/apache/datafusion/pull/12266#issuecomment-2506952118
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] closed pull request #12517: Simplify spilling merge logic
in GroupedHashAggregate
URL: https://github.com/apache/datafusion/pull/12517
--
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
Rachelint commented on issue #13275:
URL: https://github.com/apache/datafusion/issues/13275#issuecomment-2506290147
I tried the possible optimizations for`vectorized_equal_to`:
https://github.com/apache/datafusion/pull/12996#discussion_r1818601807
https://github.com/apache/datafusion/pu
goldmedal commented on code in PR #1556:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1556#discussion_r1862328322
##
derive/src/lib.rs:
##
@@ -256,3 +265,16 @@ fn visit_children(
Data::Union(_) => unimplemented!(),
}
}
+
+fn is_option(ty: &Type) ->
goldmedal commented on code in PR #1541:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1541#discussion_r1862331967
##
src/parser/mod.rs:
##
@@ -2935,12 +2935,23 @@ impl<'a> Parser<'a> {
})
} else if Token::LBracket == tok {
if di
goldmedal closed pull request #1541: Enhance the nested type access for Generic
and DuckDB dialect
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1541
--
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
goldmedal commented on PR #1541:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1541#issuecomment-2506297347
Move to another design #1551 and see
https://github.com/apache/datafusion-sqlparser-rs/pull/1541#discussion_r1861002348
for the detail.
--
This is an automated messa
Rachelint commented on issue #13275:
URL: https://github.com/apache/datafusion/issues/13275#issuecomment-2506293751
> I tried the possible optimizations for`vectorized_equal_to`: [#12996
(comment)](https://github.com/apache/datafusion/pull/12996#discussion_r1818601807)
[#12996
(comment)](h
goldmedal commented on issue #13525:
URL: https://github.com/apache/datafusion/issues/13525#issuecomment-2506337026
> I didn't mean to imply it, just that it's not something actively
tested/developed for in DataFusion. I can't recall the exact issue, but late
last year (2023) I was working
Rachelint commented on issue #13548:
URL: https://github.com/apache/datafusion/issues/13548#issuecomment-2506349949
> > I am using a M3 Macbook with 16 GB of RAM. How much RAM does your
machine have? Perhaps DataFusion only struggles with query 9 when the machine
doesn't have lots of extra
Eason0729 commented on PR #1522:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1522#issuecomment-2506522979
It seems like we reached the decision to add `recursive` instead of using
underlying dependency(stacker).
--
This is an automated message from the Apache Git Service.
Kimahriman commented on code in PR #1122:
URL: https://github.com/apache/datafusion-comet/pull/1122#discussion_r1862531038
##
spark/src/main/scala/org/apache/comet/serde/QueryPlanSerde.scala:
##
@@ -2220,6 +2220,16 @@ object QueryPlanSerde extends Logging with
ShimQueryPlanSerd
alan910127 commented on issue #13586:
URL: https://github.com/apache/datafusion/issues/13586#issuecomment-2506668979
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
Omega359 commented on code in PR #13575:
URL: https://github.com/apache/datafusion/pull/13575#discussion_r1862252991
##
datafusion/doc/src/lib.rs:
##
@@ -86,29 +90,30 @@ pub struct DocSection {
/// description: None,
/// };
///
-/// let documentation = Documen
goldmedal commented on code in PR #1556:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1556#discussion_r1862311298
##
src/ast/mod.rs:
##
@@ -7653,6 +7653,7 @@ impl fmt::Display for ShowStatementInParentType {
pub struct ShowStatementIn {
pub clause: ShowStat
findepi opened a new pull request, #13598:
URL: https://github.com/apache/datafusion/pull/13598
Temporarily pin toolchain version until problems reported by newer clippy
release are solved.
Workaround for https://github.com/apache/datafusion/issues/13597
--
This is an automated mes
iffyio commented on PR #1570:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1570#issuecomment-250558
cc @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 specific com
iffyio opened a new pull request, #1570:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1570
Fixes [clippy
warnings](https://github.com/apache/datafusion-sqlparser-rs/actions/runs/12071816871/job/33664408289?pr=1552)
on 1.83
--
This is an automated message from the Apache Gi
jonahgao closed issue #13591: Move available_parallelism() into utility function
URL: https://github.com/apache/datafusion/issues/13591
--
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
jonahgao commented on PR #13590:
URL: https://github.com/apache/datafusion/pull/13590#issuecomment-2506957606
> The rationale is that this makes the SLT test output closer to the output
a DataFusion user would typically see, in datafusion-cli, when writing float
outputs to CSV or when using
jonahgao merged PR #13595:
URL: https://github.com/apache/datafusion/pull/13595
--
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
dependabot[bot] opened a new pull request, #13588:
URL: https://github.com/apache/datafusion/pull/13588
Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.
[//]:
waynexia commented on issue #13525:
URL: https://github.com/apache/datafusion/issues/13525#issuecomment-2506407906
For the `object_store` specific problem, it doesn't have wasm support so far
as I know. I've used opendal as a workaround in
https://github.com/datafusion-contrib/datafusion-wa
iffyio commented on code in PR #1538:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1538#discussion_r1862426564
##
src/ast/mod.rs:
##
@@ -7377,15 +7420,84 @@ pub enum MySQLColumnPosition {
impl Display for MySQLColumnPosition {
fn fmt(&self, f: &mut fmt::For
demetribu commented on PR #1552:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1552#issuecomment-2506430850
@iffyio It seems the main branch is currently failing Clippy lint checks
against Rust 1.83.0.
cc @alamb
--
This is an automated message from the Apache Git Se
iffyio commented on code in PR #1565:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1565#discussion_r1862429680
##
src/ast/mod.rs:
##
@@ -5133,6 +5138,24 @@ pub enum TruncateCascadeOption {
Restrict,
}
+/// Transaction started with [ TRANSACTION | WORK ]
+
iffyio commented on code in PR #1567:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1567#discussion_r1862450915
##
tests/sqlparser_postgres.rs:
##
@@ -594,6 +594,13 @@ fn parse_alter_table_constraints_rename() {
}
}
+#[test]
+fn parse_alter_table_constrain
demetribu opened a new issue, #1569:
URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1569
It seems the main branch is currently failing Clippy lint checks against
Rust 1.83.0. Previous succ run was on stable-x86_64-unknown-linux-gnu unchanged
- rustc 1.82.0.
cc @ala
alan910127 commented on issue #13591:
URL: https://github.com/apache/datafusion/issues/13591#issuecomment-2506469259
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
alan910127 opened a new pull request, #13595:
URL: https://github.com/apache/datafusion/pull/13595
## Which issue does this PR close?
Closes #13591.
## Rationale for this change
In
https://github.com/apache/datafusion/pull/13579#pullrequestreview-2465606280,
Kimahriman commented on code in PR #1073:
URL: https://github.com/apache/datafusion-comet/pull/1073#discussion_r1862550710
##
native/spark-expr/src/list.rs:
##
@@ -413,14 +426,297 @@ impl PartialEq for GetArrayStructFields {
}
}
+#[derive(Debug, Hash)]
+pub struct ArrayI
Kimahriman commented on code in PR #1073:
URL: https://github.com/apache/datafusion-comet/pull/1073#discussion_r1862550710
##
native/spark-expr/src/list.rs:
##
@@ -413,14 +426,297 @@ impl PartialEq for GetArrayStructFields {
}
}
+#[derive(Debug, Hash)]
+pub struct ArrayI
findepi opened a new pull request, #13596:
URL: https://github.com/apache/datafusion/pull/13596
`dev/rust_lint.sh` no longer passes for me, maybe because of `rustup
update`. This is first portion of fixes suggested by clippy.
--
This is an automated message from the Apache Git Service
SemyonSinchenko commented on code in PR #1073:
URL: https://github.com/apache/datafusion-comet/pull/1073#discussion_r1862564708
##
native/spark-expr/src/list.rs:
##
@@ -413,14 +426,297 @@ impl PartialEq for GetArrayStructFields {
}
}
+#[derive(Debug, Hash)]
+pub struct A
SemyonSinchenko commented on PR #1118:
URL:
https://github.com/apache/datafusion-comet/pull/1118#issuecomment-2506605090
There is a [valid
argument](https://github.com/apache/datafusion-comet/pull/1073#discussion_r1862550710)
against it:
> The difference I think is that a LargeList can
SemyonSinchenko closed issue #1114: Spark support only i32 indexed arrays while
comet is trying to support both i32 and i64
URL: https://github.com/apache/datafusion-comet/issues/1114
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
SemyonSinchenko closed pull request #1118: chore: Make list.rs non generic &
simplify the code
URL: https://github.com/apache/datafusion-comet/pull/1118
--
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
findepi commented on PR #13598:
URL: https://github.com/apache/datafusion/pull/13598#issuecomment-2506687640
cc @Dandandan @alamb let's maybe merge this in, since the PR builds already
started to fail --
https://github.com/apache/datafusion/pull/13595#issuecomment-2506686762
--
This is
SemyonSinchenko commented on issue #1042:
URL:
https://github.com/apache/datafusion-comet/issues/1042#issuecomment-2506696139
I would like to work on `array_zip`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
findepi commented on PR #13595:
URL: https://github.com/apache/datafusion/pull/13595#issuecomment-2506686762
The
[clippy](https://github.com/apache/datafusion/actions/runs/12073608931/job/33672780242?pr=13595#logs)
job failed. This is probably nothing wrong with this PR, see
https://github
comphead commented on code in PR #13596:
URL: https://github.com/apache/datafusion/pull/13596#discussion_r1862634805
##
datafusion/functions-nested/src/concat.rs:
##
@@ -438,7 +438,7 @@ fn concat_internal(args: &[ArrayRef])
-> Result {
Ok(Arc::new(list_arr))
}
-/// Kern
alan910127 commented on PR #13595:
URL: https://github.com/apache/datafusion/pull/13595#issuecomment-2506695237
Hi @comphead, Thanks for your review! I just checked the pipeline logs, the
clippy errors are not related to my changes. Perhaps it's what @findepi
mentioned in the previous comme
Dandandan commented on code in PR #13596:
URL: https://github.com/apache/datafusion/pull/13596#discussion_r1862643735
##
datafusion/expr/src/logical_plan/display.rs:
##
@@ -181,7 +181,7 @@ impl<'a, 'b> GraphvizVisitor<'a, 'b> {
}
}
-impl<'n, 'a, 'b> TreeNodeVisitor<'n> f
Dandandan merged PR #13598:
URL: https://github.com/apache/datafusion/pull/13598
--
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
comphead commented on code in PR #13584:
URL: https://github.com/apache/datafusion/pull/13584#discussion_r1862655033
##
datafusion/core/tests/fuzz_cases/join_fuzz.rs:
##
@@ -209,6 +209,30 @@ async fn test_semi_join_1k_filtered() {
.await
}
+#[tokio::test]
+async fn test_
comphead merged PR #13565:
URL: https://github.com/apache/datafusion/pull/13565
--
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
findepi commented on PR #13596:
URL: https://github.com/apache/datafusion/pull/13596#issuecomment-2506737552
> I think we need to `#allow[missing_docs]` as the clippy now complains on
that
:) just added
--
This is an automated message from the Apache Git Service.
To respond to the
findepi commented on PR #13598:
URL: https://github.com/apache/datafusion/pull/13598#issuecomment-2506737917
thanks for the merge!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
alan910127 commented on PR #13595:
URL: https://github.com/apache/datafusion/pull/13595#issuecomment-2506747529
@comphead Rebased.
--
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
findepi commented on code in PR #13596:
URL: https://github.com/apache/datafusion/pull/13596#discussion_r1862667940
##
datafusion/physical-plan/src/sorts/sort_preserving_merge.rs:
##
@@ -746,7 +746,7 @@ mod tests {
// Split the provided record batch into multiple batch_si
comphead commented on code in PR #13596:
URL: https://github.com/apache/datafusion/pull/13596#discussion_r1862668685
##
datafusion/physical-plan/src/sorts/sort_preserving_merge.rs:
##
@@ -746,7 +746,7 @@ mod tests {
// Split the provided record batch into multiple batch_s
1 - 100 of 107 matches
Mail list logo
