ctsk commented on PR #15476:
URL: https://github.com/apache/datafusion/pull/15476#issuecomment-2764514013
Sorry about that! Thanks for tracking it down @goldmedal.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
alamb commented on issue #15422:
URL: https://github.com/apache/datafusion/issues/15422#issuecomment-2764516243
> I'd love to work on this! [@alamb](https://github.com/alamb) Could you
share the link to the blog examples please?
Most of them are linked from
https://datafusion.apache.
alamb merged PR #15496:
URL: https://github.com/apache/datafusion/pull/15496
--
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 #15149:
URL: https://github.com/apache/datafusion/pull/15149#issuecomment-2764507295
@thinkharderdev shall we merge 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 g
getChan commented on PR #15367:
URL: https://github.com/apache/datafusion/pull/15367#issuecomment-2764524744
> Thank you for the fix! I noticed there are two other tests panicked on the
same line of source code, is this fix still applicable?
>
> 1. Running sqllogictest with sqlite che
timsaucer commented on PR #1085:
URL:
https://github.com/apache/datafusion-python/pull/1085#issuecomment-2764531402
This is a really big PR and it's not immediately obvious what problem it's
trying to solve. Can you expand the description or at least link it to an issue
describing the prob
adriangb commented on code in PR #15301:
URL: https://github.com/apache/datafusion/pull/15301#discussion_r2020153605
##
datafusion/physical-expr-common/src/physical_expr.rs:
##
@@ -283,6 +284,47 @@ pub trait PhysicalExpr: Send + Sync + Display + Debug +
DynEq + DynHash {
/
suibianwanwank commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764559643
I'm not sure if I misunderstood something. The fields on both sides of the
`=` here come from two tables. I think this is similar to SELECT * FROM A t1, A
t2 where t1.id = t2.
alamb commented on code in PR #15486:
URL: https://github.com/apache/datafusion/pull/15486#discussion_r2020130924
##
datafusion/sqllogictest/test_files/timestamps.slt:
##
@@ -416,6 +416,33 @@ SELECT to_timestamp(123456789.123456789) as c1,
cast(123456789.123456789 as time
alamb commented on PR #15427:
URL: https://github.com/apache/datafusion/pull/15427#issuecomment-2764527480
> I think one issue with the current approach is that loading from env will
break.
I pushed a fix and a test for this
--
This is an automated message from the Apache Git Servi
timsaucer opened a new pull request, #1089:
URL: https://github.com/apache/datafusion-python/pull/1089
Merge 46 release into main
--
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
alamb commented on code in PR #15361:
URL: https://github.com/apache/datafusion/pull/15361#discussion_r2020139946
##
datafusion/functions/src/datetime/to_char.rs:
##
@@ -220,6 +221,13 @@ fn _to_char_scalar(
}
}
+// eagerly cast Date32 values to Date64 to supp
alamb commented on PR #13706:
URL: https://github.com/apache/datafusion/pull/13706#issuecomment-2764507105
As there isn't consensus for now, closing the PR to clear from review queue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
suibianwanwank commented on issue #15406:
URL: https://github.com/apache/datafusion/issues/15406#issuecomment-2764656937
Oops, thanks for the correction, I get it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
suibianwanwank commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764481850
I guess you might want to see this code, which outputs a True Literal at the
Join to determine if there is a matching line or not
https://github.com/apache/datafusion/blob/
2010YOUY01 commented on PR #15367:
URL: https://github.com/apache/datafusion/pull/15367#issuecomment-2764458328
Thank you for the fix! I noticed there are two other tests panicked on the
same line of source code, is this fix still applicable?
1. Running sqllogictest with sqlite check:
ht
zhuqi-lucas commented on issue #15493:
URL: https://github.com/apache/datafusion/issues/15493#issuecomment-2764477578
Revert the https://github.com/apache/datafusion/pull/15476
Just tested, also fixed the tpch bench.
--
This is an automated message from the Apache Git Service.
To r
jayzhan211 commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764483579
```
statement count 0
create table t(a int, b int) as values (11, 2), (3, 0);
statement count 0
create table t2(a int, b int) as values (11, 3), (13, 1);
quer
westhide commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020114983
##
ballista/scheduler/src/scheduler_server/grpc.rs:
##
@@ -124,14 +128,36 @@ impl SchedulerGrpc
};
let mut tasks = vec![];
+
suibianwanwank commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764487354
> In your original query, we have eq check on the same column, so I think we
can evaluate to false in `SimplifyExpressions` before
`DecorrelatePredicateSubquery`.
>
> I
jayzhan211 commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764487737
```
For query,
select e.b ,(select case when max(e2.a) > 10 then 'a' else 'b' end from t e2
where e2.b = e.b+1 ) from t e;
After `SimplifyExpressions`, we know e2.b =
milenkovicm commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020086784
##
ballista/scheduler/src/scheduler_server/grpc.rs:
##
@@ -124,14 +128,36 @@ impl SchedulerGrpc
};
let mut tasks = vec![];
westhide commented on PR #1216:
URL:
https://github.com/apache/datafusion-ballista/pull/1216#issuecomment-2764432837
> Thanks @westhide this PR makes sense,
>
> It also brings configurable validation option, which I did not really
think about. I have few questions, for which I do not
goldmedal commented on issue #15493:
URL: https://github.com/apache/datafusion/issues/15493#issuecomment-2764446051
I saw similar error messages when running tpch sqllogictest in the latest
main branch
```
~/git/datafusion ▓▒░ INCLUDE_TPCH=true cargo test --test sqllogictests --
tpch
goldmedal commented on issue #15493:
URL: https://github.com/apache/datafusion/issues/15493#issuecomment-2764449044
I guess #15476 is the root cause.
When I checked out 907150326, the tests passed.
❌ 14635dab4 (HEAD -> main, origin/main, origin/HEAD, goldmedal/main) perf:
xudong963 opened a new issue, #15495:
URL: https://github.com/apache/datafusion/issues/15495
After https://github.com/apache/datafusion/pull/15432 is merged, we'll have
partition-level statistics in `DataSource. '
To make it be used by query optimization, we should flow the info to ot
goldmedal commented on issue #15493:
URL: https://github.com/apache/datafusion/issues/15493#issuecomment-2764454319
By the way, the fail example is `dataframe`, not `custom_datasource`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
goldmedal opened a new pull request, #15496:
URL: https://github.com/apache/datafusion/pull/15496
## Which issue does this PR close?
- Closes #15493.
## Rationale for this change
## What changes are included in this PR?
- Revert #15476
#
ctsk commented on code in PR #15479:
URL: https://github.com/apache/datafusion/pull/15479#discussion_r2020104151
##
datafusion/physical-optimizer/src/coalesce_batches.rs:
##
@@ -92,3 +92,73 @@ impl PhysicalOptimizerRule for CoalesceBatches {
true
}
}
+
+/// Remove
acking-you commented on issue #15493:
URL: https://github.com/apache/datafusion/issues/15493#issuecomment-2764464827
Interesting,I just noticed that the CI also failed at this point
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
alamb commented on PR #15479:
URL: https://github.com/apache/datafusion/pull/15479#issuecomment-2764497802
BTW thank you very much @ctsk -- it is really cool to see the joins get
some careful love and attention ❤️
--
This is an automated message from the Apache Git Service.
To respond t
Omega359 commented on code in PR #15361:
URL: https://github.com/apache/datafusion/pull/15361#discussion_r2020191523
##
datafusion/functions/src/datetime/to_char.rs:
##
@@ -220,6 +221,13 @@ fn _to_char_scalar(
}
}
+// eagerly cast Date32 values to Date64 to s
Omega359 commented on code in PR #15361:
URL: https://github.com/apache/datafusion/pull/15361#discussion_r2020191523
##
datafusion/functions/src/datetime/to_char.rs:
##
@@ -220,6 +221,13 @@ fn _to_char_scalar(
}
}
+// eagerly cast Date32 values to Date64 to s
Omega359 commented on code in PR #15361:
URL: https://github.com/apache/datafusion/pull/15361#discussion_r2020170477
##
datafusion/functions/src/datetime/to_char.rs:
##
@@ -220,6 +221,13 @@ fn _to_char_scalar(
}
}
+// eagerly cast Date32 values to Date64 to s
alamb closed issue #15398: Migrate subtrait tests to `insta`
URL: https://github.com/apache/datafusion/issues/15398
--
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 unsubscri
chenkovsky commented on PR #1085:
URL:
https://github.com/apache/datafusion-python/pull/1085#issuecomment-2764593959
> This is a really big PR and it's not immediately obvious to me what
problem it's trying to solve. Can you expand the description or at least link
it to an issue describing
dependabot[bot] opened a new pull request, #1090:
URL: https://github.com/apache/datafusion-python/pull/1090
Bumps [astral-sh/setup-uv](https://github.com/astral-sh/setup-uv) from 5 to
6.
Release notes
Sourced from https://github.com/astral-sh/setup-uv/releases";>astral-sh/setup-uv
timsaucer merged PR #1081:
URL: https://github.com/apache/datafusion-python/pull/1081
--
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...
qstommyshu commented on PR #15484:
URL: https://github.com/apache/datafusion/pull/15484#issuecomment-2764550055
Sure, I'll do `git cherry-pick` and split them into multiple smaller PRs.
They should be organized by modules instead of number of lines as my commits
are mostly based on modules
xudong963 commented on PR #14955:
URL: https://github.com/apache/datafusion/pull/14955#issuecomment-2764553128
> > It seems the `statistics` generated in line 884 will be lost.
>
> >
>
> >
https://github.com/apache/datafusion/blob/main/datafusion/core/src/datasource/listing/ta
milenkovicm opened a new pull request, #1218:
URL: https://github.com/apache/datafusion-ballista/pull/1218
# Which issue does this PR close?
Closes #.
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing c
alamb commented on code in PR #15469:
URL: https://github.com/apache/datafusion/pull/15469#discussion_r2020125627
##
datafusion/physical-plan/src/sorts/sort.rs:
##
@@ -397,16 +393,13 @@ impl ExternalSorter {
self.metrics.spill_metrics.spill_file_count.value()
}
-
suibianwanwank commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764503913
> ```
>After `SimplifyExpressions`, we know e2.b = 2.b + 1 is false, since they
are the same column. And, then we can infer taht >max(e2.a) is NULL.
>select e.b ,(select
alamb commented on PR #15484:
URL: https://github.com/apache/datafusion/pull/15484#issuecomment-2764504770
> The code changes is now done, please review carefully as the code changes
is LARGE.
Thank you so much @qstommyshu -- this looks epic
Is there any chance you can
alamb commented on PR #15423:
URL: https://github.com/apache/datafusion/pull/15423#issuecomment-2764511786
FYI @ctsk this seems potentially related to some of the work you are doing
as well
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
alamb closed issue #15493: `custom_datasource` example panicked during
`RepartitionExec` planning
URL: https://github.com/apache/datafusion/issues/15493
--
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
alan910127 commented on issue #15492:
URL: https://github.com/apache/datafusion/issues/15492#issuecomment-2764511135
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
edmondop commented on issue #15422:
URL: https://github.com/apache/datafusion/issues/15422#issuecomment-2764530536
I see two possible strategies:
1. build first a self contained example (i.e. datafusion-examples crate)
inspired by one the blog posts, and store in datafusion-examples
timsaucer merged PR #1089:
URL: https://github.com/apache/datafusion-python/pull/1089
--
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 merged PR #1088:
URL: https://github.com/apache/datafusion-python/pull/1088
--
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 PR #14955:
URL: https://github.com/apache/datafusion/pull/14955#issuecomment-2764527928
> It seems the `statistics` generated in line 884 will be lost.
>
>
https://github.com/apache/datafusion/blob/main/datafusion/core/src/datasource/listing/table.rs#L884-L960
adriangb commented on code in PR #15301:
URL: https://github.com/apache/datafusion/pull/15301#discussion_r2020157010
##
datafusion/proto/src/physical_plan/to_proto.rs:
##
@@ -210,6 +212,7 @@ pub fn serialize_physical_expr(
value: &Arc,
codec: &dyn PhysicalExtensionCode
suibianwanwank commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764502074
> > Hi @xudong963, would you be interested in reviewing this PR? Any
feedback would be greatly appreciated!
>
> I'm sorry, it's weird I didn't receive the notification,
adriangb commented on code in PR #15301:
URL: https://github.com/apache/datafusion/pull/15301#discussion_r2020153780
##
datafusion/physical-plan/src/dynamic_filters.rs:
##
@@ -0,0 +1,226 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor li
alamb commented on PR #15496:
URL: https://github.com/apache/datafusion/pull/15496#issuecomment-2764507735
Since CI is failing without this let's merge it in
--
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
alamb commented on code in PR #15427:
URL: https://github.com/apache/datafusion/pull/15427#discussion_r2020134939
##
datafusion-cli/tests/cli_integration.rs:
##
@@ -74,6 +75,31 @@ fn cli_quick_test<'a>(
assert_cmd_snapshot!(cmd);
}
+#[rstest]
Review Comment:
Thanks,
barsela1 closed issue #1778: Support for MySQL := assignment operator
URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1778
--
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
berkaysynnada commented on PR #15411:
URL: https://github.com/apache/datafusion/pull/15411#issuecomment-2751437750
I've two questions: 1) Why don't we use BatchDeserializer to deserialize? 2)
Why are we converting them to async as they don't yield at all already?
--
This is an automated m
adriangb commented on code in PR #15301:
URL: https://github.com/apache/datafusion/pull/15301#discussion_r2020152758
##
datafusion/physical-plan/src/topk/mod.rs:
##
@@ -186,6 +235,90 @@ impl TopK {
Ok(())
}
+fn calculate_dynamic_filters(
+thresholds:
qstommyshu opened a new pull request, #15497:
URL: https://github.com/apache/datafusion/pull/15497
## Which issue does this PR close?
- Related #15397, this is a part of #15484 breaking down.
## Rationale for this change
## What changes are included in thi
milenkovicm commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020223746
##
ballista/scheduler/src/state/mod.rs:
##
@@ -248,7 +262,7 @@ impl SchedulerState,` and
`task.manager.launch_multi_task` and cancel the jobs there?
milenkovicm commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020224499
##
ballista/scheduler/src/state/task_manager.rs:
##
@@ -524,24 +524,35 @@ impl TaskManager
pub(crate) async fn launch_multi_task(
&self,
alamb commented on code in PR #15481:
URL: https://github.com/apache/datafusion/pull/15481#discussion_r2020122550
##
docs/source/library-user-guide/profiling.md:
##
@@ -82,6 +82,43 @@ CARGO_PROFILE_RELEASE_DEBUG=true cargo flamegraph --root
--bench sql_planner --
[Video: how
alamb closed pull request #13706: Document SQL dialect guidance
URL: https://github.com/apache/datafusion/pull/13706
--
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
jatin510 commented on code in PR #15486:
URL: https://github.com/apache/datafusion/pull/15486#discussion_r2020135443
##
datafusion/sqllogictest/test_files/timestamps.slt:
##
@@ -416,6 +416,33 @@ SELECT to_timestamp(123456789.123456789) as c1,
cast(123456789.123456789 as time
-
thinkharderdev commented on PR #15149:
URL: https://github.com/apache/datafusion/pull/15149#issuecomment-2764520144
> @thinkharderdev shall we merge this PR?
I think it's good to go. There are still some open comments so I was waiting
for another approval, but if we're all agreed then
timsaucer commented on PR #1072:
URL:
https://github.com/apache/datafusion-python/pull/1072#issuecomment-2764532709
I recommend closing this PR or at lest changing it to:
```python
global_ctx = SessionContext.global_ctx()
```
That *I think* will also help with making sure
jayzhan211 commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764506178
when we compare table1.a = table2.b, we compare them row by row.
```
t1.a t2.b
1 1
2 3
3 2
4 4
```
```
t1.a=t2.b
true
false
false
true
Max-Meldrum commented on issue #15422:
URL: https://github.com/apache/datafusion/issues/15422#issuecomment-2764551007
> > I'd love to work on this! [@alamb](https://github.com/alamb) Could you
share the link to the blog examples please?
>
> Most of them are linked from
https://datafu
westhide commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020169629
##
ballista/scheduler/src/scheduler_server/grpc.rs:
##
@@ -124,14 +128,36 @@ impl SchedulerGrpc
};
let mut tasks = vec![];
+
qstommyshu closed pull request #15484: Migrate datafusion/sql tests to
insta,sql integrations
URL: https://github.com/apache/datafusion/pull/15484
--
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
kurosch commented on issue #1044:
URL:
https://github.com/apache/datafusion-comet/issues/1044#issuecomment-2764660850
I would like to work on the `[]` operator. I assume it has to be implemented
for arrays and maps.
--
This is an automated message from the Apache Git Service.
To respond
jayzhan211 commented on PR #15281:
URL: https://github.com/apache/datafusion/pull/15281#issuecomment-2764486169
In your original query, we have eq check on the same column, so I think we
can evaluate to false in `SimplifyExpressions` before
`DecorrelatePredicateSubquery`.
In my query
timsaucer opened a new pull request, #65:
URL: https://github.com/apache/datafusion-site/pull/65
Work in progress
--
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 unsubsc
leoyvens commented on code in PR #15473:
URL: https://github.com/apache/datafusion/pull/15473#discussion_r2020240967
##
datafusion/datasource/benches/split_groups_by_statistics.rs:
##
@@ -0,0 +1,178 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more co
renato2099 commented on PR #65:
URL: https://github.com/apache/datafusion-site/pull/65#issuecomment-2764724217
looks like a lot was done for this release! thanks for putting this together
Tim!
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
comphead commented on code in PR #15481:
URL: https://github.com/apache/datafusion/pull/15481#discussion_r2020242855
##
docs/source/library-user-guide/profiling.md:
##
@@ -82,6 +82,43 @@ CARGO_PROFILE_RELEASE_DEBUG=true cargo flamegraph --root
--bench sql_planner --
[Video:
comphead opened a new issue, #15498:
URL: https://github.com/apache/datafusion/issues/15498
### Is your feature request related to a problem or challenge?
I was following to https://lucumr.pocoo.org/2025/3/23/from-string/ blog
where a cheaper way introduced when converting a `String`
comphead commented on issue #15498:
URL: https://github.com/apache/datafusion/issues/15498#issuecomment-2764731939
In DataFusion there are some places where conversion used by indiredctions,
or `as_str` or other methods, I'm thinking of creating a string utility method
to make this conversi
clflushopt commented on issue #14608:
URL: https://github.com/apache/datafusion/issues/14608#issuecomment-2764733544
@lmwnshn @matthewmturner we now have a live crate for integrations
https://crates.io/crates/tpchgen and a cli available
https://github.com/clflushopt/tpchgen-rs special thank
TheBuilderJR commented on PR #15295:
URL: https://github.com/apache/datafusion/pull/15295#issuecomment-2764720923
I tried patching this in and getting these failures. Any idea why?
Unfortunately I can't share the full source.
```
Error fetching table metadata: Failed to collect dat
comphead commented on issue #15406:
URL: https://github.com/apache/datafusion/issues/15406#issuecomment-2764735186
Thanks for taking care on that, the SMJ structure is historically too
complicated and break it down into smaller ones would benefit.
For example it can be broken down in
comphead commented on code in PR #1550:
URL: https://github.com/apache/datafusion-comet/pull/1550#discussion_r2020248798
##
spark/src/main/scala/org/apache/spark/sql/comet/CometScanExec.scala:
##
@@ -490,8 +490,7 @@ object CometScanExec extends DataTypeSupport {
// TODO a
marvelshan commented on issue #15309:
URL: https://github.com/apache/datafusion/issues/15309#issuecomment-2764874334
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
milenkovicm commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020086784
##
ballista/scheduler/src/scheduler_server/grpc.rs:
##
@@ -124,14 +128,36 @@ impl SchedulerGrpc
};
let mut tasks = vec![];
tomershaniii commented on PR #1747:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1747#issuecomment-2764615971
Hi again @iffyio @mvzink, see the latest commit, per your suggestions:
- Permissive parsing (removed value validation for key/value parameters)
- Centralized all
xudong963 commented on code in PR #15473:
URL: https://github.com/apache/datafusion/pull/15473#discussion_r2020323603
##
datafusion/datasource/benches/split_groups_by_statistics.rs:
##
@@ -0,0 +1,178 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
CrystalZhou0529 commented on PR #1033:
URL:
https://github.com/apache/datafusion-python/pull/1033#issuecomment-2764914770
This could be reviewed now!
--
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
westhide commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020090087
##
ballista/scheduler/src/scheduler_server/grpc.rs:
##
@@ -124,14 +128,36 @@ impl SchedulerGrpc
};
let mut tasks = vec![];
+
comphead merged PR #1550:
URL: https://github.com/apache/datafusion-comet/pull/1550
--
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...@d
github-actions[bot] commented on PR #12523:
URL: https://github.com/apache/datafusion/pull/12523#issuecomment-2764926021
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 #14308:
URL: https://github.com/apache/datafusion/pull/14308#issuecomment-2764925938
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
alamb commented on issue #14608:
URL: https://github.com/apache/datafusion/issues/14608#issuecomment-2764746130
I think the next question in my mind is exactly how to integrate this into
datafusion-cli
We could follow the model of duckdb and create a table function like
`dbgen(sf = 1
berkaysynnada commented on code in PR #15489:
URL: https://github.com/apache/datafusion/pull/15489#discussion_r2020254263
##
datafusion/core/src/dataframe/mod.rs:
##
@@ -724,6 +764,45 @@ impl DataFrame {
})
}
+/// Calculate the union of two [`DataFrame`]s usi
alamb merged PR #15497:
URL: https://github.com/apache/datafusion/pull/15497
--
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
Omega359 commented on code in PR #15489:
URL: https://github.com/apache/datafusion/pull/15489#discussion_r2020279541
##
datafusion/core/src/dataframe/mod.rs:
##
@@ -724,6 +764,45 @@ impl DataFrame {
})
}
+/// Calculate the union of two [`DataFrame`]s using co
qstommyshu opened a new pull request, #15499:
URL: https://github.com/apache/datafusion/pull/15499
## Which issue does this PR close?
- Related #15397, this is a part of #15484 breaking down.
- Checkout things to note of the whole migration in comments section of
#15484.
xudong963 commented on PR #15432:
URL: https://github.com/apache/datafusion/pull/15432#issuecomment-2764431052
> Would it be possible to add some unit tests for
`compute_summary_statistics`? Something like:
Thanks @alamb ! I'm cooking it
--
This is an automated message from the Apa
westhide commented on code in PR #1212:
URL:
https://github.com/apache/datafusion-ballista/pull/1212#discussion_r2020084368
##
ballista/scheduler/src/scheduler_server/grpc.rs:
##
@@ -124,14 +128,36 @@ impl SchedulerGrpc
};
let mut tasks = vec![];
+
goldmedal commented on PR #15423:
URL: https://github.com/apache/datafusion/pull/15423#issuecomment-2764422078
> I think to support this selection vector, the executors need to be updated
to interpret an additional metadata column. However, since executors are part
of the public interface t
1 - 100 of 147 matches
Mail list logo
