ozankabak commented on code in PR #10650:
URL: https://github.com/apache/datafusion/pull/10650#discussion_r1614446423
##
datafusion/physical-expr/src/expressions/cast.rs:
##
@@ -170,7 +170,8 @@ impl PhysicalExpr for CastExpr {
let target_type = &self.cast_type;
ozankabak commented on PR #10650:
URL: https://github.com/apache/datafusion/pull/10650#issuecomment-2131118238
Maybe I can help clarify things a little bit. The purpose of the PR is not
to change the current fallback behavior, just simplify its implementation.
AFAICT @berkaysynnada is movin
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614462365
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
ozankabak commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614484172
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&self
ozankabak commented on PR #10590:
URL: https://github.com/apache/datafusion/pull/10590#issuecomment-2131154079
The aim has been to make the `COPY` and `CREATE EXTERNAL TABLE` statements
use the same `OPTIONS` syntax. The PR you mention was actually a part of
multi-PR effort to make this so
goldmedal opened a new pull request, #10661:
URL: https://github.com/apache/datafusion/pull/10661
## Which issue does this PR close?
Closes #10658
## Rationale for this change
## What changes are included in this PR?
## Are these changes te
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614503459
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614503459
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614503459
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
alamb commented on code in PR #10648:
URL: https://github.com/apache/datafusion/pull/10648#discussion_r1614514043
##
datafusion/physical-expr-common/src/utils.rs:
##
@@ -100,15 +103,37 @@ pub fn reverse_order_bys(order_bys: &[PhysicalSortExpr])
-> Vec`.
+/// If conversion is no
sonhmai opened a new pull request, #470:
URL: https://github.com/apache/datafusion-comet/pull/470
## Which issue does this PR close?
Closes #463
## Rationale for this change
Fix for substring with negative indices produces incorrect results
## What changes are
alamb commented on issue #8708:
URL: https://github.com/apache/datafusion/issues/8708#issuecomment-2131167747
After https://github.com/apache/datafusion/pull/10648 and
https://github.com/apache/datafusion/issues/10389 I think we have a pretty good
set of examples of how to move aggregates o
alamb commented on PR #10645:
URL: https://github.com/apache/datafusion/pull/10645#issuecomment-2131168957
> The gate issue is not related to the PR.. could we rerun?
I restarted it
FYI another trick we have found is if you close the PR and then reopen it,
github will rerun al
advancedxy commented on code in PR #449:
URL: https://github.com/apache/datafusion-comet/pull/449#discussion_r1614516643
##
core/src/execution/datafusion/expressions/scalar_funcs/hex.rs:
##
@@ -0,0 +1,371 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or m
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614522462
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
alamb opened a new pull request, #10662:
URL: https://github.com/apache/datafusion/pull/10662
## Which issue does this PR close?
Closes #.
## Rationale for this change
When running `cargo build` locally I see
```
warning:
/Users/andrewlamb/Software/data
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614522462
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614522462
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
alamb commented on PR #10627:
URL: https://github.com/apache/datafusion/pull/10627#issuecomment-2131171739
> If this is the case then there is divergence between postgres and
arrow-rs. Which takes priority?
I would personally suggest we do whatever is consistent with arrow (and
easie
sonhmai commented on PR #470:
URL: https://github.com/apache/datafusion-comet/pull/470#issuecomment-2131173613
@viirya @andygrove can you take a look. 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 abov
alamb commented on code in PR #10268:
URL: https://github.com/apache/datafusion/pull/10268#discussion_r1614530863
##
datafusion/sqllogictest/test_files/select.slt:
##
@@ -1473,7 +1473,7 @@ DROP TABLE t;
# related to https://github.com/apache/datafusion/issues/8814
statement
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614503459
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614522462
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
advancedxy commented on code in PR #455:
URL: https://github.com/apache/datafusion-comet/pull/455#discussion_r1614532699
##
spark/src/test/scala/org/apache/comet/CometExpressionCoverageSuite.scala:
##
@@ -217,6 +325,25 @@ class CometExpressionCoverageSuite extends CometTestBase
alamb commented on code in PR #10646:
URL: https://github.com/apache/datafusion/pull/10646#discussion_r1614534288
##
datafusion/substrait/src/variation_const.rs:
##
@@ -37,3 +38,58 @@ pub const DEFAULT_CONTAINER_TYPE_REF: u32 = 0;
pub const LARGE_CONTAINER_TYPE_REF: u32 = 1;
p
alamb merged PR #10657:
URL: https://github.com/apache/datafusion/pull/10657
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@datafusi
alamb closed issue #10656: Wrong error thrown when unnesting a list of struct
URL: https://github.com/apache/datafusion/issues/10656
--
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 merged PR #10645:
URL: https://github.com/apache/datafusion/pull/10645
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@datafusi
alamb closed issue #10605: Incorrect statistics read for binary columns in
parquet
URL: https://github.com/apache/datafusion/issues/10605
--
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 commented on code in PR #10655:
URL: https://github.com/apache/datafusion/pull/10655#discussion_r1614536504
##
datafusion/functions/src/core/getfield.rs:
##
@@ -106,6 +106,9 @@ impl ScalarUDFImpl for GetFieldFunc {
};
let access_schema = GetFieldAccessSch
alamb closed issue #10654: Error on `NULL["field_name"]`: The expression to get
an indexed field is only valid for `List`, `Struct`, or `Map` types, got Null
URL: https://github.com/apache/datafusion/issues/10654
--
This is an automated message from the Apache Git Service.
To respond to the m
alamb commented on PR #10655:
URL: https://github.com/apache/datafusion/pull/10655#issuecomment-2131186851
Thank you for the speedy review @jayzhan211
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to g
alamb merged PR #10655:
URL: https://github.com/apache/datafusion/pull/10655
--
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
goldmedal commented on code in PR #10661:
URL: https://github.com/apache/datafusion/pull/10661#discussion_r1614539494
##
datafusion/optimizer/src/push_down_limit.rs:
##
@@ -65,7 +65,6 @@ impl OptimizerRule for PushDownLimit {
};
let Limit { skip, fetch, input
goldmedal commented on code in PR #10661:
URL: https://github.com/apache/datafusion/pull/10661#discussion_r1614539989
##
datafusion/core/src/execution/context/mod.rs:
##
@@ -2893,21 +2893,21 @@ mod tests {
for function in all_default_functions() {
let udf =
goldmedal commented on code in PR #10661:
URL: https://github.com/apache/datafusion/pull/10661#discussion_r1614539989
##
datafusion/core/src/execution/context/mod.rs:
##
@@ -2893,21 +2893,21 @@ mod tests {
for function in all_default_functions() {
let udf =
alamb commented on code in PR #10644:
URL: https://github.com/apache/datafusion/pull/10644#discussion_r1614539590
##
datafusion/functions-aggregate/src/median.rs:
##
@@ -15,71 +15,105 @@
// specific language governing permissions and limitations
// under the License.
-//! #
alamb merged PR #10632:
URL: https://github.com/apache/datafusion/pull/10632
--
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 code in PR #10661:
URL: https://github.com/apache/datafusion/pull/10661#discussion_r1614546056
##
datafusion/core/src/execution/context/mod.rs:
##
@@ -2860,6 +2863,57 @@ mod tests {
Ok(())
}
+#[tokio::test]
+async fn test_register_defau
alamb commented on PR #10661:
URL: https://github.com/apache/datafusion/pull/10661#issuecomment-2131195735
Thank you @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 specific co
alamb commented on issue #10639:
URL: https://github.com/apache/datafusion/issues/10639#issuecomment-2131196191
Thanks @jamesmcm -- this would be a good first issue I think for someone
with Rust experience looking to help with DataFusion. The protobuf
serialization code is a bit hairy I fi
alamb commented on issue #10631:
URL: https://github.com/apache/datafusion/issues/10631#issuecomment-2131197268
> I see TaskContext is just part of SessionState. Could we just make
TaskContext wrap SessionState?
I can't remember why TaskContext doesn't wrap SessionState -- maybe
@tu
tustvold commented on issue #10631:
URL: https://github.com/apache/datafusion/issues/10631#issuecomment-2131201943
IIRC SessionConfig is the static configuration used to create a
SessionContext, which is an interior mutable wrapper around SessionState.
The idea was a query is planned
jonahgao merged PR #10662:
URL: https://github.com/apache/datafusion/pull/10662
--
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
alamb opened a new issue, #10663:
URL: https://github.com/apache/datafusion/issues/10663
### Is your feature request related to a problem or challenge?
Part of https://github.com/apache/datafusion/issues/9726 to complete the
LogialPlan --> SQL conversion
Converting `LogicalPlan
alamb opened a new issue, #10664:
URL: https://github.com/apache/datafusion/issues/10664
### Is your feature request related to a problem or challenge?
Part of https://github.com/apache/datafusion/issues/9726 to complete the
LogialPlan --> SQL conversion
Converting `LogicalPlan
alamb commented on issue #10652:
URL: https://github.com/apache/datafusion/issues/10652#issuecomment-2131211386
> Is there any plan to support them?
I think we should make plans to support them!
I started collecting issues on
https://github.com/apache/datafusion/issues/8661
alamb merged PR #10623:
URL: https://github.com/apache/datafusion/pull/10623
--
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 #10623:
URL: https://github.com/apache/datafusion/pull/10623#issuecomment-2131213174
Thanks everyone for the reviews!
--
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
alamb commented on issue #2326:
URL: https://github.com/apache/datafusion/issues/2326#issuecomment-2131214534
> I added an issue to support recursive unnest: #10660, i think it shoul
belong to this epic
Added
--
This is an automated message from the Apache Git Service.
To respond t
Abdullahsab3 commented on issue #10602:
URL: https://github.com/apache/datafusion/issues/10602#issuecomment-2131223669
Thanks for filing the ticket and for all the detailed explanations! very
enriching
I wonder whether the Postgres behavior is actually that bad. Though it looks
weird
timsaucer commented on issue #715:
URL:
https://github.com/apache/datafusion-python/issues/715#issuecomment-2131229316
My statement above about testing on rust side is likely incorrect. I ran the
same test above but loading the dataframe from a parquet file instead of
creating in memory an
alamb commented on issue #7306:
URL: https://github.com/apache/datafusion/issues/7306#issuecomment-2131234075
> What do you think we should do? I can add that paragraph, but maybe we
also want to link the examples? What else do you think it's needed to close this
Those both sound like
alamb commented on PR #10650:
URL: https://github.com/apache/datafusion/pull/10650#issuecomment-2131235103
> Maybe we can do the following: Instead of making every
TableProviderFactory responsible for implementing this behavior in create, we
can add some general mechanism at the trait level
alamb commented on PR #10573:
URL: https://github.com/apache/datafusion/pull/10573#issuecomment-2131235504
> This shouldn't have passed checks.
>
> ```
> + cargo fmt --all -- --check
> `cargo metadata` exited with an error: error: failed to load manifest for
workspace member `/o
alamb commented on PR #10640:
URL: https://github.com/apache/datafusion/pull/10640#issuecomment-2131235829
Thanks @Blizzara and @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 spec
alamb commented on code in PR #10634:
URL: https://github.com/apache/datafusion/pull/10634#discussion_r1614628009
##
datafusion/sqllogictest/test_files/aggregate.slt:
##
@@ -4374,6 +4374,42 @@ GROUP BY dummy
text1, text1, text1
+# Tests for aggregating with NaN values
+
goldmedal commented on PR #10661:
URL: https://github.com/apache/datafusion/pull/10661#issuecomment-2131238078
Thanks, @alamb!
Actually, I tried to add some tests for `MemoryFunctionRegistry` before I
fixed it, but I found everything was fine. I guess the reason is that we don't
inse
jayzhan211 opened a new pull request, #10665:
URL: https://github.com/apache/datafusion/pull/10665
## Which issue does this PR close?
Closes #.
## Rationale for this change
## What changes are included in this PR?
## Are these changes tested
andygrove commented on code in PR #461:
URL: https://github.com/apache/datafusion-comet/pull/461#discussion_r1614638361
##
core/src/execution/datafusion/expressions/cast.rs:
##
@@ -622,14 +590,89 @@ impl Cast {
self.eval_mode,
from_type,
andygrove commented on code in PR #461:
URL: https://github.com/apache/datafusion-comet/pull/461#discussion_r1614639502
##
core/src/execution/datafusion/expressions/cast.rs:
##
@@ -622,14 +590,89 @@ impl Cast {
self.eval_mode,
from_type,
andygrove merged PR #716:
URL: https://github.com/apache/datafusion-python/pull/716
--
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
jayzhan211 commented on code in PR #10644:
URL: https://github.com/apache/datafusion/pull/10644#discussion_r1614640283
##
datafusion/functions-aggregate/Cargo.toml:
##
@@ -39,6 +39,7 @@ path = "src/lib.rs"
[dependencies]
arrow = { workspace = true }
+arrow-schema = { workspa
dependabot[bot] opened a new pull request, #717:
URL: https://github.com/apache/datafusion-python/pull/717
Bumps [syn](https://github.com/dtolnay/syn) from 2.0.63 to 2.0.66.
Release notes
Sourced from https://github.com/dtolnay/syn/releases";>syn's
releases.
2.0.66
Al
dependabot[bot] closed pull request #706: build(deps): bump syn from 2.0.63 to
2.0.64
URL: https://github.com/apache/datafusion-python/pull/706
--
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
dependabot[bot] commented on PR #706:
URL:
https://github.com/apache/datafusion-python/pull/706#issuecomment-2131243514
Superseded by #717.
--
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
alamb commented on PR #10662:
URL: https://github.com/apache/datafusion/pull/10662#issuecomment-2131244635
> oops.. my mistake. :P Many thanks.
No worries!
--
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 code in PR #10268:
URL: https://github.com/apache/datafusion/pull/10268#discussion_r1614654486
##
datafusion/sqllogictest/test_files/joins.slt:
##
@@ -1778,7 +1778,7 @@ AS VALUES
('BB', 6, 1);
query TII
-select col1, col2, coalesce(sum_col3, 0) as sum_col3
alamb commented on issue #10602:
URL: https://github.com/apache/datafusion/issues/10602#issuecomment-2131254591
Thank you @tustvold and @Abdullahsab3 and @mhilton and @appletreeisyellow
for the thoughts.
From my perspective, the current (non timezone aware) `date_bin` function
has t
peter-toth opened a new pull request, #10666:
URL: https://github.com/apache/datafusion/pull/10666
The idea of using slices to return a node's children came up here:
https://github.com/apache/datafusion/pull/10543#discussion_r1614057653.
While it is not possible in all `TreeNode` implemen
alamb merged PR #10665:
URL: https://github.com/apache/datafusion/pull/10665
--
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
jayzhan211 commented on code in PR #10268:
URL: https://github.com/apache/datafusion/pull/10268#discussion_r1614656536
##
datafusion/sqllogictest/test_files/joins.slt:
##
@@ -1778,7 +1778,7 @@ AS VALUES
('BB', 6, 1);
query TII
-select col1, col2, coalesce(sum_col3, 0) as sum
jayzhan211 commented on code in PR #10268:
URL: https://github.com/apache/datafusion/pull/10268#discussion_r1614656536
##
datafusion/sqllogictest/test_files/joins.slt:
##
@@ -1778,7 +1778,7 @@ AS VALUES
('BB', 6, 1);
query TII
-select col1, col2, coalesce(sum_col3, 0) as sum
peter-toth commented on code in PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#discussion_r1614657172
##
datafusion/physical-plan/src/work_table.rs:
##
@@ -169,7 +169,7 @@ impl ExecutionPlan for WorkTableExec {
&self.cache
}
-fn children(&sel
jayzhan211 commented on issue #8708:
URL: https://github.com/apache/datafusion/issues/8708#issuecomment-2131265763
> After #10648 and #10389 I think we have a pretty good set of examples of
how to move aggregates out of the core (thanks to all the foundations layed by
@jayzhan211 )
>
jonahgao commented on code in PR #10531:
URL: https://github.com/apache/datafusion/pull/10531#discussion_r1614666495
##
datafusion/substrait/src/logical_plan/producer.rs:
##
@@ -512,6 +572,62 @@ pub fn to_substrait_rel(
}
}
+fn to_substrait_named_struct(schema: &DFSchema
alamb commented on code in PR #10647:
URL: https://github.com/apache/datafusion/pull/10647#discussion_r1614666750
##
datafusion/core/src/datasource/physical_plan/parquet/mod.rs:
##
@@ -642,11 +714,22 @@ fn should_enable_page_index(
.unwrap_or(false)
}
-/// Factor
alamb commented on PR #10543:
URL: https://github.com/apache/datafusion/pull/10543#issuecomment-2131268529
Awesome!
I plan to merge this PR tomorrow unless anyone else would like time to
review.
--
This is an automated message from the Apache Git Service.
To respond to the message
jayzhan211 opened a new issue, #10667:
URL: https://github.com/apache/datafusion/issues/10667
### Is your feature request related to a problem or challenge?
Part of #8708
1. Move variance aggregate expression in
`datafusion/physical-expr/src/aggregate/variance.rs` to `functions-ag
jonahgao commented on code in PR #10531:
URL: https://github.com/apache/datafusion/pull/10531#discussion_r1614671030
##
datafusion/substrait/tests/cases/roundtrip_logical_plan.rs:
##
@@ -685,6 +685,19 @@ async fn roundtrip_literal_struct() -> Result<()> {
.await
}
+#[tok
jayzhan211 opened a new issue, #10668:
URL: https://github.com/apache/datafusion/issues/10668
### Is your feature request related to a problem or challenge?
Similar to #10667 but for Variance Population function
### Describe the solution you'd like
_No response_
##
vaibhawvipul opened a new pull request, #471:
URL: https://github.com/apache/datafusion-comet/pull/471
## Which issue does this PR close?
Closes #465 .
## Rationale for this change
## What changes are included in this PR?
## How are these ch
jonahgao commented on code in PR #10531:
URL: https://github.com/apache/datafusion/pull/10531#discussion_r1614673020
##
datafusion/substrait/tests/cases/roundtrip_logical_plan.rs:
##
@@ -685,6 +685,19 @@ async fn roundtrip_literal_struct() -> Result<()> {
.await
}
+#[tok
edmondop commented on issue #7306:
URL: https://github.com/apache/datafusion/issues/7306#issuecomment-2131289288
> > What do you think we should do? I can add that paragraph, but maybe we
also want to link the examples? What else do you think it's needed to close this
>
> Those both s
goldmedal commented on PR #10619:
URL: https://github.com/apache/datafusion/pull/10619#issuecomment-2131302454
On a related note, I found that the usage flow of ObjectStore in DataFusion
involves registering the source as a table and then querying the table. This
approach makes sense. Howev
DDtKey opened a new issue, #10669:
URL: https://github.com/apache/datafusion/issues/10669
### Is your feature request related to a problem or challenge?
CSV writers usually supports configuration of quote style/mode with the
following options:
- `Always`
- `Necessary`
- `Neve
DDtKey opened a new issue, #10670:
URL: https://github.com/apache/datafusion/issues/10670
### Describe the bug
quote` parameter of CSV writer is not getting passed to `arrow-csv` writer:
https://github.com/apache/datafusion/blob/ea92ae72f7ec2e941d35aa077c6a39f74523ab63/datafusion/c
andygrove opened a new pull request, #472:
URL: https://github.com/apache/datafusion-comet/pull/472
## Which issue does this PR close?
N/A
## Rationale for this change
Comet Fuzz is a standalone project for generating random data and queries
and executing
andygrove commented on code in PR #461:
URL: https://github.com/apache/datafusion-comet/pull/461#discussion_r1614788921
##
core/src/execution/datafusion/expressions/cast.rs:
##
@@ -622,14 +590,89 @@ impl Cast {
self.eval_mode,
from_type,
andygrove commented on code in PR #461:
URL: https://github.com/apache/datafusion-comet/pull/461#discussion_r1614789127
##
core/src/execution/datafusion/expressions/cast.rs:
##
@@ -622,14 +590,89 @@ impl Cast {
self.eval_mode,
from_type,
DDtKey opened a new pull request, #10671:
URL: https://github.com/apache/datafusion/pull/10671
## Which issue does this PR close?
Closes #10670
## Rationale for this change
## What changes are included in this PR?
## Are these changes tested
duongcongtoai opened a new issue, #10672:
URL: https://github.com/apache/datafusion/issues/10672
### Describe the bug
Given this query
```
select unnest(struct_c1c0), unnest(list_c2c0) from (
select unnest(column1) as struct_c1c0, unnest(column2)['c0'] as
list_c2c
duongcongtoai commented on issue #10672:
URL: https://github.com/apache/datafusion/issues/10672#issuecomment-2131375997
I'm taking it
--
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
DDtKey commented on issue #10669:
URL: https://github.com/apache/datafusion/issues/10669#issuecomment-2131388179
I think this might be labeled with `good first issue`, there are links to
the code that needs to be changed and it is also possible to write
`sqllogictest` similar to https://git
andygrove merged PR #461:
URL: https://github.com/apache/datafusion-comet/pull/461
--
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...@da
viirya opened a new pull request, #473:
URL: https://github.com/apache/datafusion-comet/pull/473
## Which issue does this PR close?
Closes #.
## Rationale for this change
## What changes are included in this PR?
## How are these changes test
viirya opened a new issue, #474:
URL: https://github.com/apache/datafusion-comet/issues/474
### Describe the bug
During debugging some test failures in #437, I found that `CometScanExec`
returns empty dictionary values after first batch in failed queries. It is
because in `Data.impor
viirya commented on PR #473:
URL: https://github.com/apache/datafusion-comet/pull/473#issuecomment-2131403965
cc @sunchao
--
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
viirya commented on code in PR #473:
URL: https://github.com/apache/datafusion-comet/pull/473#discussion_r1614819511
##
common/src/main/java/org/apache/comet/parquet/ColumnReader.java:
##
@@ -230,15 +230,15 @@ public CometDecodedVector loadVector() {
// return plain vec
comphead commented on code in PR #10647:
URL: https://github.com/apache/datafusion/pull/10647#discussion_r1614830679
##
datafusion/core/src/datasource/physical_plan/parquet/schema_adapter.rs:
##
@@ -20,35 +20,38 @@ use arrow_schema::{Schema, SchemaRef};
use std::fmt::Debug;
us
1 - 100 of 119 matches
Mail list logo
