Skip to content

Commit ec39ee6

Browse files
findepifa-assistant
authored andcommitted
Fix failure when computing Snowflake distinct array_agg
This changes how Snowflake array_agg is implemented. Before the change it was implemented via 'simplify', but this has the limitation that aggregated values need to be the same as sorting values when distinct is on (otherwise results would not be well-defined). The new implementation delegates at the accumulator level and casts to VARIANT at the aggregation finalize stage. For this to work, some changes are backported in DataFusion fork. GitOrigin-RevId: 5bf1370d844aba5df3fa59d7ddc2b5a4efc84bab
1 parent 65a6e4c commit ec39ee6

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ version = "2.0.0-beta.28"
4444
# numbers as crates.io / the upstream repository.
4545
# See https://doc.rust-lang.org/cargo/reference/overriding-dependencies.html#the-patch-section
4646
[patch.crates-io]
47-
datafusion = { git = "https://github.com/sdf-labs/arrow-datafusion", rev = "fba71744de5a40aed493afb3ef9b810a89cbe6fc" } # @ sdf/45 branch
47+
datafusion = { git = "https://github.com/sdf-labs/arrow-datafusion", rev = "7fb794e2776d0601328c359aaa6a47c8d94deffc" } # @ sdf/45
4848
# OR, when working locally
4949
#datafusion = { path = "../arrow-datafusion/datafusion/core" }
5050

0 commit comments

Comments
 (0)