iffyio closed issue #1607: Parser rejects column aliases that Snowflake accepts
URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1607
--
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
iffyio commented on issue #1607:
URL:
https://github.com/apache/datafusion-sqlparser-rs/issues/1607#issuecomment-2646091587
Done in https://github.com/apache/datafusion-sqlparser-rs/pull/1632
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
shehabgamin commented on code in PR #14440:
URL: https://github.com/apache/datafusion/pull/14440#discussion_r1948008922
##
datafusion/expr-common/src/signature.rs:
##
@@ -225,6 +229,45 @@ impl Display for TypeSignatureClass {
}
}
+impl TypeSignatureClass {
+/// Retur
iffyio commented on code in PR #1653:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1653#discussion_r1948012742
##
src/parser/mod.rs:
##
@@ -9092,6 +9092,16 @@ impl<'a> Parser<'a> {
});
}
}
+if self.di
iffyio merged PR #1713:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1713
--
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
chenkovsky commented on PR #14362:
URL: https://github.com/apache/datafusion/pull/14362#issuecomment-2646050299
because in this PR system columns are propagated by default. I guess we have
to write a lot of monkey patch to prevent system column leak.
--
This is an automated message from t
chenkovsky commented on PR #14362:
URL: https://github.com/apache/datafusion/pull/14362#issuecomment-2646049486
could you please also test
```rust
let tb = ctx.table("test").await.unwrap();
tb.write_csv(".csv", DataFrameWriteOptions::new(),
None).await.unwrap();
```
clflushopt commented on issue #3843:
URL: https://github.com/apache/datafusion/issues/3843#issuecomment-2646048358
Small update; I started looking at the initial join selection rule
implemented by backtracking from the reference to `HashBuildProbeOrder` and did
a review over the PR that int
github-actions[bot] closed pull request #13607: [POC] Fuse operations in
`equal_rows_arr`
URL: https://github.com/apache/datafusion/pull/13607
--
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
duongcongtoai commented on issue #14554:
URL: https://github.com/apache/datafusion/issues/14554#issuecomment-2646013487
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 comme
codecov-commenter commented on PR #1380:
URL:
https://github.com/apache/datafusion-comet/pull/1380#issuecomment-2645976629
##
[Codecov](https://app.codecov.io/gh/apache/datafusion-comet/pull/1380?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_ca
gstvg commented on issue #14205:
URL: https://github.com/apache/datafusion/issues/14205#issuecomment-2645957917
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
logan-keede commented on PR #14555:
URL: https://github.com/apache/datafusion/pull/14555#issuecomment-2645932154
> No worries at all - when we merge to main everything in each PR is
squashed into a single commit.
Thanks. I know but still, it feels off.
This is ready for review,
mitsuhiko commented on code in PR #1713:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1713#discussion_r1947940772
##
tests/sqlparser_duckdb.rs:
##
@@ -246,6 +246,67 @@ fn test_create_table_macro() {
assert_eq!(expected, macro_);
}
+#[test]
+fn test_select
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1947939409
##
datafusion/expr/src/udf.rs:
##
@@ -951,4 +952,10 @@ The following regular expression functions are
supported:"#,
label: "Other Functions",
desc
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1947937272
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,260 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
Omega359 opened a new pull request, #14561:
URL: https://github.com/apache/datafusion/pull/14561
## Which issue does this PR close?
- Closes #14560
## Rationale for this change
json parity with bytes impl
## What changes are included in this PR?
Code
Omega359 commented on issue #14560:
URL: https://github.com/apache/datafusion/issues/14560#issuecomment-2645910034
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 u
Omega359 opened a new issue, #14560:
URL: https://github.com/apache/datafusion/issues/14560
### Is your feature request related to a problem or challenge?
Currently there is support for logical plan -> to/from bytes with an
extension codec but there is support for the same for json. T
andygrove commented on PR #1369:
URL:
https://github.com/apache/datafusion-comet/pull/1369#issuecomment-2645907012
I tried running TPC-H locally with this PR but there is excessive spilling
and the benchmark is extremely slow. q3 took 3.4 minutes compared to 16 seconds
on main branch.
andygrove opened a new issue, #1381:
URL: https://github.com/apache/datafusion-comet/issues/1381
### What is the problem the feature request solves?
See comment at
https://github.com/apache/datafusion-comet/pull/1329#discussion_r1944106936
### Describe the potential solution
andygrove commented on code in PR #1329:
URL: https://github.com/apache/datafusion-comet/pull/1329#discussion_r1947927557
##
native/core/src/execution/jni_api.rs:
##
@@ -233,11 +242,12 @@ pub unsafe extern "system" fn
Java_org_apache_comet_Native_createPlan(
stream
andygrove merged PR #1329:
URL: https://github.com/apache/datafusion-comet/pull/1329
--
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 closed issue #1328: Improve performance of update metrics
URL: https://github.com/apache/datafusion-comet/issues/1328
--
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.
berkaysynnada closed issue #14505: Do We Need partition_by Field in
(Bounded)WindowAggExec ?
URL: https://github.com/apache/datafusion/issues/14505
--
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 th
berkaysynnada merged PR #14526:
URL: https://github.com/apache/datafusion/pull/14526
--
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...@
erenavsarogullari commented on PR #14417:
URL: https://github.com/apache/datafusion/pull/14417#issuecomment-2645880016
Thanks @jayzhan211 and @findepi for the reviews.
Updated this PR in terms of previous feedback from `array_max` PR:
https://github.com/apache/datafusion/pull/14470
Pl
alamb commented on PR #14555:
URL: https://github.com/apache/datafusion/pull/14555#issuecomment-2645872676
> apologies, there were some major fumbles, this should not have taken 11+
commits, I can open a separate PR if this would look too ugly.
No worries at all - when we merge to mai
logan-keede commented on PR #14555:
URL: https://github.com/apache/datafusion/pull/14555#issuecomment-2645871731
apologies, there were some major fumbles, this should not have taken 11+
commits,
I can open a separate PR if this would look too ugly.
--
This is an automated message from
jkosh44 commented on code in PR #14532:
URL: https://github.com/apache/datafusion/pull/14532#discussion_r1947908046
##
datafusion/expr-common/src/signature.rs:
##
@@ -227,25 +226,11 @@ impl Display for TypeSignatureClass {
#[derive(Debug, Clone, PartialEq, Eq, PartialOrd, Has
andygrove commented on code in PR #1329:
URL: https://github.com/apache/datafusion-comet/pull/1329#discussion_r1947907141
##
native/core/src/execution/jni_api.rs:
##
@@ -233,11 +242,12 @@ pub unsafe extern "system" fn
Java_org_apache_comet_Native_createPlan(
stream
milenkovicm commented on PR #1176:
URL:
https://github.com/apache/datafusion-ballista/pull/1176#issuecomment-2645856481
converting to draft until #1173 and #1172 are closed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
milenkovicm opened a new pull request, #1176:
URL: https://github.com/apache/datafusion-ballista/pull/1176
# Which issue does this PR close?
Closes #.
# Rationale for this change
keep up to date with datafusion releases
# What changes are included in this
ugoa commented on issue #14557:
URL: https://github.com/apache/datafusion/issues/14557#issuecomment-2645846496
Please help review workaround solution in
https://github.com/apache/datafusion/pull/14544/commits/997504f85e37ef6269e5ee660a1136031d6d2a2
in https://github.com/apache/datafusion/pu
ugoa commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947900039
##
docs/source/library-user-guide/query-optimizer.md:
##
@@ -187,94 +235,101 @@ name to be used in a schema, `display_name` should be
used.
### Utilities
-There
ugoa commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947898504
##
docs/source/library-user-guide/query-optimizer.md:
##
@@ -187,94 +235,101 @@ name to be used in a schema, `display_name` should be
used.
### Utilities
-There
ugoa commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947898504
##
docs/source/library-user-guide/query-optimizer.md:
##
@@ -187,94 +235,101 @@ name to be used in a schema, `display_name` should be
used.
### Utilities
-There
irenjj commented on PR #14526:
URL: https://github.com/apache/datafusion/pull/14526#issuecomment-2645837400
> LGTM @irenjj, thank you. I've sent a minor commit for further
improvements. Do they make sense to you?
Sure, thank you! @berkaysynnada
--
This is an automated message from
ugoa commented on PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#issuecomment-2645835876
So I managed to strip off the setup code with some regex trick in Python
`docs/rustdoc_trim.py`, and now the doc shows clean snippets while the doc-test
still able to pass.
A few poi
kazantsev-maksim commented on code in PR #1343:
URL: https://github.com/apache/datafusion-comet/pull/1343#discussion_r1947893539
##
spark/src/main/scala/org/apache/comet/serde/QueryPlanSerde.scala:
##
@@ -2387,6 +2387,8 @@ object QueryPlanSerde extends Logging with
ShimQueryPla
kazantsev-maksim commented on code in PR #1343:
URL: https://github.com/apache/datafusion-comet/pull/1343#discussion_r1947892464
##
spark/src/main/scala/org/apache/comet/serde/arrays.scala:
##
@@ -126,6 +126,21 @@ object CometArraysOverlap extends CometExpressionSerde
with Inco
geoffreyclaude commented on issue #9415:
URL: https://github.com/apache/datafusion/issues/9415#issuecomment-2645830852
I've opened https://github.com/apache/datafusion/pull/14547 to launch the
discussion. PR was simpler than expected as all new tasks spawned do so in the
context of a tokio
kazantsev-maksim commented on code in PR #1343:
URL: https://github.com/apache/datafusion-comet/pull/1343#discussion_r1947891630
##
spark/src/test/scala/org/apache/comet/CometArrayExpressionSuite.scala:
##
@@ -292,4 +292,89 @@ class CometArrayExpressionSuite extends CometTestBas
alamb commented on PR #14483:
URL: https://github.com/apache/datafusion/pull/14483#issuecomment-2645809803
🚀 let's give it a try
--
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
dependabot[bot] opened a new pull request, #14559:
URL: https://github.com/apache/datafusion/pull/14559
Bumps [nix](https://github.com/nix-rust/nix) from 0.28.0 to 0.29.0.
Changelog
Sourced from https://github.com/nix-rust/nix/blob/master/CHANGELOG.md";>nix's
changelog.
[0.2
alamb merged PR #14483:
URL: https://github.com/apache/datafusion/pull/14483
--
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 #14483:
URL: https://github.com/apache/datafusion/pull/14483#issuecomment-2645809871
Thanks @mbrobbel and @findepi
--
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
alamb closed issue #14135: Discuss: Check in Cargo.lock file?
URL: https://github.com/apache/datafusion/issues/14135
--
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
alamb merged PR #14556:
URL: https://github.com/apache/datafusion/pull/14556
--
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 #14556:
URL: https://github.com/apache/datafusion/pull/14556#issuecomment-2645809640
Thank you @jonahgao
--
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 #14549: Extended tests are failing on main
URL: https://github.com/apache/datafusion/issues/14549
--
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
andygrove opened a new pull request, #1380:
URL: https://github.com/apache/datafusion-comet/pull/1380
## Which issue does this PR close?
Part of https://github.com/apache/datafusion-comet/issues/1345
## Rationale for this change
Refactor in preparation for
alamb commented on code in PR #14440:
URL: https://github.com/apache/datafusion/pull/14440#discussion_r1947837053
##
datafusion/expr-common/src/signature.rs:
##
@@ -127,12 +132,11 @@ pub enum TypeSignature {
Exact(Vec),
/// One or more arguments belonging to the [`Type
jonahgao commented on issue #14380:
URL: https://github.com/apache/datafusion/issues/14380#issuecomment-2645780946
> Hmm, my immediate thought here is that if we cannot guarantee correctly
coerced inputs during the first pass, then it's probably not worth attempting
to coerce in the builder
jonahgao commented on issue #14380:
URL: https://github.com/apache/datafusion/issues/14380#issuecomment-2645778218
> The original discussion about wildcard expansion is [#11639
(comment)](https://github.com/apache/datafusion/issues/11639#issuecomment-2250895868).
It's used to delay the wild
ugoa commented on PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#issuecomment-2645752534
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 unsubscrib
berkaysynnada commented on issue #14514:
URL: https://github.com/apache/datafusion/issues/14514#issuecomment-2645748627
> I noticed [#7622](https://github.com/apache/datafusion/pull/7622), if the
syntax was originally not supported by
[sqlparser](https://github.com/apache/datafusion-sqlpars
cj-zhukov opened a new pull request, #14558:
URL: https://github.com/apache/datafusion/pull/14558
## Which issue does this PR close?
- Closes #14548.
## Rationale for this change
## What changes are included in this PR?
## Are these changes
ugoa commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947837198
##
docs/source/library-user-guide/adding-udfs.md:
##
@@ -55,62 +55,138 @@ of arguments.
This a lower level API with more functionality but is more complex, also
doc
alamb commented on PR #14555:
URL: https://github.com/apache/datafusion/pull/14555#issuecomment-2645741135
> can I get some help with wasm ci? I have hit a wall there.
Looks like that has now been resolved, but there are some other issues that
remain
--
This is an automated message
alamb commented on issue #14557:
URL: https://github.com/apache/datafusion/issues/14557#issuecomment-2645737409
> [@alamb](https://github.com/alamb) assign this to me
Thanks @bhavesh26
Note in the future you can put the comment `take` and the bot will
automatically assign it t
alamb commented on PR #14483:
URL: https://github.com/apache/datafusion/pull/14483#issuecomment-2645735179
I merged up from main to resolve a conflict and applied the suggestion. I
plan to merge this once we get a clean CI run
--
This is an automated message from the Apache Git Service.
T
alamb commented on PR #14543:
URL: https://github.com/apache/datafusion/pull/14543#issuecomment-2645733546
Thanks again @logan-keede
--
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
iffyio opened a new pull request, #1718:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1718
Adds support for the `BEGIN ... EXCEPTION ... END` syntax in BigQuery:
```sql
BEGIN
SELECT 1;
SELECT 2;
EXCEPTION WHEN ERROR THEN
SELECT 3;
SELECT 4;
E
bhavesh26 commented on issue #14557:
URL: https://github.com/apache/datafusion/issues/14557#issuecomment-2645713479
@alamb assign this to 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 above to go to the sp
pmcgleenon commented on issue #14246:
URL: https://github.com/apache/datafusion/issues/14246#issuecomment-2645694266
Makes sense 👍
I've created a PR https://github.com/ClickHouse/ClickBench/pull/304
--
This is an automated message from the Apache Git Service.
To respond to the mes
timsaucer opened a new pull request, #54:
URL: https://github.com/apache/datafusion-site/pull/54
This PR adds in a blog post announcing `datafusion-python` 44.0.0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
chenkovsky commented on PR #14362:
URL: https://github.com/apache/datafusion/pull/14362#issuecomment-2645665768
1. could you please add more dataframe tests?
e.g.
```
assert!(ctx.table("test").await.unwrap().union(ctx.table("test").await.unwrap()).unwrap().select(vec![col("_rowi
alamb commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947818087
##
docs/source/library-user-guide/adding-udfs.md:
##
@@ -55,62 +55,138 @@ of arguments.
This a lower level API with more functionality but is more complex, also
do
LucaCappelletti94 commented on code in PR #1707:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1707#discussion_r1947817780
##
src/ast/ddl.rs:
##
@@ -1030,20 +1030,29 @@ impl fmt::Display for KeyOrIndexDisplay {
/// [1]: https://dev.mysql.com/doc/refman/8.0/en/cre
alamb commented on code in PR #1707:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1707#discussion_r1947816903
##
src/ast/ddl.rs:
##
@@ -1030,20 +1030,29 @@ impl fmt::Display for KeyOrIndexDisplay {
/// [1]: https://dev.mysql.com/doc/refman/8.0/en/create-table.ht
alamb commented on code in PR #1707:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1707#discussion_r1947815817
##
src/ast/ddl.rs:
##
@@ -1030,20 +1030,29 @@ impl fmt::Display for KeyOrIndexDisplay {
/// [1]: https://dev.mysql.com/doc/refman/8.0/en/create-table.ht
alamb commented on PR #1712:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1712#issuecomment-2645599573
> See my comments inline. Generally speaking, I support parsing of arbitrary
options using DataLoadingOptions but would rename and move this struct to a
more central locati
alamb merged PR #14320:
URL: https://github.com/apache/datafusion/pull/14320
--
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 #14320:
URL: https://github.com/apache/datafusion/pull/14320#issuecomment-2645597947
Thanks @comphead and @Omega359
--
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 spec
alamb commented on PR #14528:
URL: https://github.com/apache/datafusion/pull/14528#issuecomment-2645596984
Thanks again @Chen-Yuan-Lai
--
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
alamb merged PR #14528:
URL: https://github.com/apache/datafusion/pull/14528
--
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
ugoa commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947806611
##
docs/source/library-user-guide/adding-udfs.md:
##
@@ -55,62 +55,138 @@ of arguments.
This a lower level API with more functionality but is more complex, also
doc
ugoa commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947806611
##
docs/source/library-user-guide/adding-udfs.md:
##
@@ -55,62 +55,138 @@ of arguments.
This a lower level API with more functionality but is more complex, also
doc
logan-keede commented on PR #14555:
URL: https://github.com/apache/datafusion/pull/14555#issuecomment-2645589511
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 comment.
T
alamb commented on issue #14246:
URL: https://github.com/apache/datafusion/issues/14246#issuecomment-2645571982
> If it looks reasonable, I can create a ClickBench PR to update the results
It does look reasonable to me. Thank you @pmcgleenon 🙏
--
This is an automated messa
alamb commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947793559
##
docs/source/library-user-guide/query-optimizer.md:
##
@@ -187,94 +235,101 @@ name to be used in a schema, `display_name` should be
used.
### Utilities
-Ther
alamb commented on code in PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#discussion_r1947719701
##
docs/source/library-user-guide/adding-udfs.md:
##
@@ -145,6 +220,24 @@ This "works" in isolation, i.e. if you have a slice of
`ArrayRef`s, you can call
`ArrayRe
alamb commented on issue #14557:
URL: https://github.com/apache/datafusion/issues/14557#issuecomment-2645372507
I think this would be a good first issue as it is super helpful, well
specified, and doesn't need deep DataFusion internals expertise
--
This is an automated message from the Ap
alamb opened a new issue, #14557:
URL: https://github.com/apache/datafusion/issues/14557
### Is your feature request related to a problem or challenge?
Part of https://github.com/apache/datafusion/issues/7013
@ugoa has updated our examples in the following PR to all compile and
alamb commented on PR #14485:
URL: https://github.com/apache/datafusion/pull/14485#issuecomment-2645369634
> > Although one major problem I have with using # to hide setup code, is
that the sphinx won't respect the leading # and still renders the whole block,
which leading huge chunk of ugl
iffyio opened a new pull request, #1717:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1717
Adds support for the `EXECUTE IMMEDIATE` statement
```sql
EXECUTE IMMEDIATE 'SELECT 1' USING 2 AS foo;
```
[BigQuery](https://cloud.google.com/bigquery/docs/reference
alamb opened a new pull request, #14556:
URL: https://github.com/apache/datafusion/pull/14556
## Which issue does this PR close?
- Closes https://github.com/apache/datafusion/issues/14549
## Rationale for this change
This currently fails on main:
```
INCLUDE_SQLITE=
alamb commented on PR #6:
URL: https://github.com/apache/datafusion-testing/pull/6#issuecomment-2645291208
Thank you @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 specific commen
logan-keede commented on PR #14555:
URL: https://github.com/apache/datafusion/pull/14555#issuecomment-2645220572
can I get some help with wasm ci? I have hit a wall there.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
ugoa closed pull request #14485: Run test from `adding-udfs.md` documentation
URL: https://github.com/apache/datafusion/pull/14485
--
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
ugoa commented on PR #14544:
URL: https://github.com/apache/datafusion/pull/14544#issuecomment-2645043123
Hi @alamb I have passed all the doc-test in `library-user-guide` folder, the
one part left is in the `query-optimizer.md`, the previous code snippets are
outdated and I cannot find alte
ugoa commented on PR #14485:
URL: https://github.com/apache/datafusion/pull/14485#issuecomment-2645007165
Closing this PR as I have moved the changes to [PR
14544](https://github.com/apache/datafusion/pull/14544)
--
This is an automated message from the Apache Git Service.
To respond to t
chenkovsky commented on PR #14057:
URL: https://github.com/apache/datafusion/pull/14057#issuecomment-2644943280
for #14362 ,if we want to make it feasible, I think at least it should check
metadata dict loaded from file for every format. (e.g. parquet), otherwise the
whole system may be bro
iffyio commented on PR #1716:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1716#issuecomment-2644891227
FYI @goldmedal
--
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
iffyio opened a new pull request, #1716:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1716
Continuation of #1551
This moves over two of the special case compound expressions delimited by
period to use the new `CompoundFieldAccess` representation.
It moves the log
2010YOUY01 closed issue #12338: Use `StringViewArray` as output of `substr`
when input was `StringArray`
URL: https://github.com/apache/datafusion/issues/12338
--
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
2010YOUY01 merged PR #14498:
URL: https://github.com/apache/datafusion/pull/14498
--
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
2010YOUY01 commented on PR #14498:
URL: https://github.com/apache/datafusion/pull/14498#issuecomment-2644818671
> My benchmark results show no change in Q22 (but also clearly the data is
quite noisy 🤔 )
>
> ```
>
> Benchmark tpch_sf1.json
> --
logan-keede opened a new pull request, #14555:
URL: https://github.com/apache/datafusion/pull/14555
## Which issue does this PR close?
- part of https://github.com/apache/datafusion/issues/14462
- Part of https://github.com/apache/datafusion/issues/1.
## Rationale
1 - 100 of 113 matches
Mail list logo
