Skip to content

[FEATURE] Add Integration tests for explain with pushdown disabled in Calcite #3785

Open
@yuancu

Description

@yuancu

Is your feature request related to a problem?

Currently, all integration tests for the EXPLAIN command are limited to scenarios where pushdown is enabled. For example, in ExplainIT.java:

String expected =
    isCalciteEnabled()
        ? loadFromFile("expectedOutput/calcite/explain_output.json")
        : loadFromFile("expectedOutput/ppl/explain_output.json");

Both cases rely on execution with pushdown enabled, leaving a gap in test coverage.

What solution would you like?
Add new EXPLAIN tests that explicitly run with pushdown disabled, to verify the correctness and completeness of the explain output when fallback to Calcite occurs.

What alternatives have you considered?

Do you have any additional context?
#3777

Metadata

Metadata

Assignees

No one assigned

    Labels

    calcitecalcite migration releatedenhancementNew feature or requesttestingRelated to improving software testing

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions