Skip to content

Misc refactoring parsing - #950

Merged
pgiraud merged 11 commits into
masterfrom
misc_refactoring_parsing
Aug 26, 2026
Merged

Misc refactoring parsing#950
pgiraud merged 11 commits into
masterfrom
misc_refactoring_parsing

Conversation

@pgiraud

@pgiraud pgiraud commented Aug 17, 2026

Copy link
Copy Markdown
Member

No description provided.

@pgiraud
pgiraud force-pushed the misc_refactoring_parsing branch 8 times, most recently from fcc2ac3 to a085b50 Compare August 19, 2026 11:51
@pgiraud
pgiraud force-pushed the misc_refactoring_parsing branch 2 times, most recently from 7edfaf7 to 8097cec Compare August 25, 2026 08:14
pgiraud added 11 commits August 25, 2026 16:55
Instead use a revised field for that.
Same for ACTUAL_STARTUP_TIME.
We now search for files with `-plan` suffix and determine the
corresponding `-expect` file.
Right|Left Semi|Anti

In the tests, I had to manually delete some keys in the expect file
because it's impossible to determine them from the text format. It's the
case for Inner Unique key which appears systematically in the JSON
format but doesn't appear in text.
InitPlan and SubPlan relationship is already set earlier.
For other cases, for:
 - Join nodes (Nested Loop, Merge Join, Hash Join), the first child is
   "Outer", the others are "Inner",
 - Append and Bitmap, relationship is "Member",
 - Subquery Scan, relationship is "Subquery".

A new file is added to from-text list. A SQL script is added to generate
both Text and JSON format plans. However, the JSON plan needed to be
adapted a bit.
We also improve parsing of Group|Hash|Sort Key to possibly handle
Grouping Sets.
@pgiraud
pgiraud force-pushed the misc_refactoring_parsing branch from 8097cec to 575f701 Compare August 26, 2026 11:58
@pgiraud
pgiraud merged commit bd2b097 into master Aug 26, 2026
3 checks passed
@pgiraud
pgiraud deleted the misc_refactoring_parsing branch August 26, 2026 12:09
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