Skip to content

Conversation

@aafemt
Copy link
Contributor

@aafemt aafemt commented Jan 5, 2026

An improvement for EXEC_PATH_DISPLAY. It is more convenient to see the execution tree in ISQL instead of debug log only.

XML formatting of the nodes wasn't changed from original implementation.

@asfernandes
Copy link
Member

The XML node-tree is a development feature with missing things.
I'm against put it in production in the current state.
Also it prints internal details (numeric constants) that we do not want to stabilize.

@aafemt
Copy link
Contributor Author

aafemt commented Jan 6, 2026

So does BLR printer. This feature is for debugging, not production.

@asfernandes
Copy link
Member

BLR is stablished thing, it's API.
Emitted BLR should always be parseable and there were already BLR-to-text filters of stored routines.

@aafemt
Copy link
Contributor Author

aafemt commented Jan 6, 2026

Note: this pull request doesn't change the node printer, just add a new way to receive the result. If the node printer is incomplete and messy - it is a separate issue and should be fixed by separate PR(s).

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.

2 participants