Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Join transfer made more robust to nullable unique constraints in complex right children #667

Merged
merged 2 commits into from Jun 10, 2023
Merged

Join transfer made more robust to nullable unique constraints in complex right children #667

merged 2 commits into from Jun 10, 2023

Conversation

bcogrel
Copy link
Member

@bcogrel bcogrel commented Jun 10, 2023

Now optimizes examples like

LJ
   EXTENSIONAL TABLE1(0:a,1:d)
   LJ
      EXTENSIONAL TABLE7(0:a,1:c)
      EXTENSIONAL TABLE7(0:a,2:b)

into

LJ
   EXTENSIONAL TABLE1(0:a,1:d)
   EXTENSIONAL TABLE7(0:a,1:c,2:b)

and

LJ IS_NOT_NULL(a)
   EXTENSIONAL TABLE1(0:d)
   LJ
      EXTENSIONAL TABLE7(0:a,1:c)
      EXTENSIONAL TABLE7(0:a,2:b)

into

LJ IS_NOT_NULL(a)
   EXTENSIONAL TABLE1(0:d)
   EXTENSIONAL TABLE7(0:a,1:c,2:b)

where the first column of TABLE7 has a nullable unique constraint.

@bcogrel bcogrel added this to the v5.1.0 milestone Jun 10, 2023
@bcogrel bcogrel merged commit 3eb5c7c into version5 Jun 10, 2023
19 checks passed
@bcogrel bcogrel deleted the feature/join-transfer-nullable-uc-nested branch June 10, 2023 07:09
@bcogrel bcogrel changed the title Join transfer made more robust to non-nullable unique constraints in complex right children Join transfer made more robust to nullable unique constraints in complex right children Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
status: fixed type: enhancement type: performance
Projects
None yet
Milestone
v5.1.0
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@bcogrel