Skip to content

Conversation

lnkuiper
Copy link
Contributor

@lnkuiper lnkuiper commented Sep 17, 2024

Fixes:

  1. https://github.com/duckdblabs/duckdb-internal/issues/3041 - Recurse into materialized CTE definitions in FilterPushdown.
  2. Binder error when manually materializing a CTE #13850 - Disable auto-CTE materialization when one or more of the CTEs in the query are manually materialized.
  3. Json property name with special characters produce inconsistent results with json -> 'propertyname' and json_extract #13948 - Use JSONPath instead of JSONPointer to resolve JSON extracts that do no explicitly begin with a $ or /, and also implement a special case for when the extract argument is integral (array extract). This more closely matches PostgreSQL behavior.
  4. INTERNAL Error: Attempted to dereference unique_ptr that is NULL #13938 - Reset chunk in PhysicalPieceWiseMergeJoin if it has a cardinality of 0 (it may have been sliced before, changing the vectors to Dictionary, which messes up a Reference when filling Left/Outer join NULLs).

I've also tweaked the CardinalityEstimator to fix a regression to TPC-H Q9 introduced when going from 1.0.0 to 1.1.0.

EDIT: I've also tweaked some allocation settings so that we are less likely to OOM when running large queries on small hardware.

@duckdb-draftbot duckdb-draftbot marked this pull request as draft September 18, 2024 10:35
@lnkuiper lnkuiper marked this pull request as ready for review September 18, 2024 10:35
@Mytherin Mytherin merged commit f9e96b1 into duckdb:main Sep 18, 2024
42 of 43 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Sep 27, 2024
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Sep 27, 2024
Bugfixes (duckdb/duckdb#13982)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants