Skip to content

[SQL] Reduce dataflow graph json file produced by default#5864

Merged
mihaibudiu merged 2 commits intomainfrom
issue5813
Mar 20, 2026
Merged

[SQL] Reduce dataflow graph json file produced by default#5864
mihaibudiu merged 2 commits intomainfrom
issue5813

Conversation

@mihaibudiu
Copy link
Contributor

There are two commits.
The first one
Fixes #5813

By default the compiler verbosity flag -v will control the detail of the dataflow graph produced. The default verbosity used by the pipeline manager will omit the JSON serialization of the Calcite expressions, significantly shrinking the plan size.

The second commit cleans up some of the ignored tests: some are no longer ignored, some we will never support are deleted.

@mihaibudiu mihaibudiu enabled auto-merge March 18, 2026 23:24
@mihaibudiu mihaibudiu requested a review from lalithsuresh March 18, 2026 23:24
Copy link

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mihaibudiu mihaibudiu added this pull request to the merge queue Mar 19, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 19, 2026
@mihaibudiu mihaibudiu added this pull request to the merge queue Mar 19, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 19, 2026
…bosity and omit Calcite expressions

Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
@mihaibudiu mihaibudiu enabled auto-merge March 19, 2026 22:10
@mihaibudiu mihaibudiu added this pull request to the merge queue Mar 19, 2026
Merged via the queue into main with commit 418f138 Mar 20, 2026
1 check passed
@mihaibudiu mihaibudiu deleted the issue5813 branch March 20, 2026 00:12
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.

[SQL] Shrink down the dataflow.json generated by the compiler

3 participants