Apply Coralogix v38 changes onto v39 DataFusion #343
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add pool_size method to MemoryPool * Fix * Fmt Co-authored-by: Daniël Heres <[email protected]>
avantgardnerio
force-pushed
the
bg_39
branch
from
c871c9f to
f16c370
Compare
avantgardnerio
force-pushed
the
bg_39
branch
from
30e77f5 to
00c7b70
Compare
avantgardnerio
force-pushed
the
bg_39
branch
from
00c7b70 to
491821e
Compare
Co-authored-by: svranesevic <[email protected]>
…ache#11301) * AggregateExec: Take grouping sets into account for InputOrderMode * pr comments # Conflicts: # datafusion/physical-plan/src/aggregates/mod.rs
* Add customizable equality and hash functions to UDFs * Improve equals and hash_value documentation * Add tests for parameterized UDFs * CI issues
* Add extension hooks for encoding and decoding UDAFs and UDWFs * Add tests for encoding and decoding UDAFs
* Merge * Update datafusion/physical-expr/src/aggregate/approx_percentile_cont.rs Co-authored-by: Georgi Krastev <[email protected]> --------- Co-authored-by: Georgi Krastev <[email protected]>
* Fix nullability of output * Change test to use nullable
Less opportunities for bugs
* Fix size calculation in ArrayAggGroupsAccumulator * Respect `ignore_nulls` flag in `DistinctArrayAgg` * Respect `ignore_nulls` in `ArrayAggAccumulator` * Respect `ignore_nulls` in `ArrayAggGroupsAccumulator` Also add an `ArrayAgg` groups accumulator for boolean arrays
Looks like it depends on the Rust version so it fails in DQE
* Minor optimization of the null handling in ArrayAggAccumulator * Add `ignore_nulls` flag to `AggrFn`
* Revert "Backport workaround for union is (not) null (#265)" This reverts commit dd7ca08. * Reapply union workaround * Reapply perf change --------- Co-authored-by: Samuel Colvin <[email protected]> Co-authored-by: Andy Grove <[email protected]>
* Support encoding and decoding UnnestExec * test: check record count and types in parquet window test (apache#12277) * test: check record count and types in paqruet window test * Review suggestions Co-authored-by: Andrew Lamb <[email protected]> --------- Co-authored-by: Andrew Lamb <[email protected]> --------- Co-authored-by: Eduard Karacharov <[email protected]> Co-authored-by: Andrew Lamb <[email protected]>
* Add state for distributed joins * Make DistributedJoinState public * Implement shared bitmap hook * make trait public * remove unused method * make enum public * Get probe threads from distributed state * Avoid issue with probe threads counter wrapping * Use naming conventions from upstream PR * Add hook to register metrics * Add partition to register_metrics
* ignore writer shutdown error * cargo check
Add CoalescePartitionsExec (#272)
avantgardnerio
force-pushed
the
bg_39
branch
from
b410c88 to
a71cb72
Compare
Cherry pick upstream fix (#274)
If all UDF arguments are scalars, so should be the result. In most cases, such function calls will be contant-folded, however if for whatever reason the are not optimized, we want to avoid an error due to array length mismatch.
…2965) (#276) This was done in apache#12922 only for math functions. We now generalize this fallback to all scalar UDFs.
* Try and fix swap_hash_join * Only swap projections when join does not have projections * just backport upstream fix * remove println
* Support Duration in min/max agg functions * Attempt to fix build * Attempt to fix build - Fix chrono version * Revert "Attempt to fix build - Fix chrono version" This reverts commit fd76fe6. * Revert "Attempt to fix build" This reverts commit 9114b86. --------- Co-authored-by: svranesevic <[email protected]>
avantgardnerio
force-pushed
the
bg_39
branch
from
72f2c7f to
58eb9f2
Compare
* Drop rust-toolchain * Fix panics in array_union * Fix the chrono
Co-authored-by: svranesevic <[email protected]>
) * fix: remove the Sized requirement on ExecutionPlan::name() Signed-off-by: Ruihang Xia <[email protected]> * Update datafusion/physical-plan/src/lib.rs Co-authored-by: Andrew Lamb <[email protected]> * add document and test Signed-off-by: Ruihang Xia <[email protected]> --------- Signed-off-by: Ruihang Xia <[email protected]> Co-authored-by: Andrew Lamb <[email protected]> (cherry picked from commit a19fc62)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
Closes #.
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?