FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Citus: skip INHERITS generation until citusdata/citus#8553 is fixed by mrigger · Pull Request #1343 · sqlancer/sqlancer · GitHub

Citus: skip INHERITS generation until citusdata/citus#8553 is fixed - #1343

Merged
mrigger merged 1 commit into
mainfrom
fix/citus-skip-inherits-8553
May 2, 2026
Merged

Citus: skip INHERITS generation until citusdata/citus#8553 is fixed#1343
mrigger merged 1 commit into
mainfrom
fix/citus-skip-inherits-8553

Conversation

mrigger commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Citus's distributed planner returns wrong results when an inheritance parent is cross-joined with a distributed table inside a LEFT JOIN ... ON FALSE and a WHERE filters on the parent column, which the TLP-WHERE oracle keeps tripping over (~1 in 8 Citus CI runs). Make PostgresTableGenerator.generateInherits() protected so CitusTableGenerator can override it as a no-op while the new CitusBugs.bug8553 flag is set.

Citus's distributed planner returns wrong results when an inheritance
parent is cross-joined with a distributed table inside a
LEFT JOIN ... ON FALSE and a WHERE filters on the parent column,
which the TLP-WHERE oracle keeps tripping over (~1 in 8 Citus CI runs).
Make PostgresTableGenerator.generateInherits() protected so
CitusTableGenerator can override it as a no-op while the new
CitusBugs.bug8553 flag is set.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
mrigger merged commit 3024850 into main May 2, 2026
24 of 25 checks passed
mrigger deleted the fix/citus-skip-inherits-8553 branch May 2, 2026 05:46
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL