Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SPARK-47939][SQL] Implement a new Analyzer rule to move ParameterizedQuery inside ExplainCommand and DescribeQueryCommand #46209

Closed
wants to merge 4 commits into from
Closed

[SPARK-47939][SQL] Implement a new Analyzer rule to move ParameterizedQuery inside ExplainCommand and DescribeQueryCommand #46209

wants to merge 4 commits into from

Commits on Apr 26, 2024

  1. Add tests for parameterized describe and explain

    @vladimirg-db
    vladimirg-db committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    50af3fa View commit details
    Browse the repository at this point in the history

  2. Implement a new Analyzer rule for SupervisingCommands 

    Mark DescribeQueryCommand and ExplainCommand as SupervisingCommand (they don't expose their wrapped nodes, but supervise them internally). Move ParameterizedQuery into SupervisingCommand for parameters to be resolved correctly.
    @vladimirg-db
    vladimirg-db committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    070043b View commit details
    Browse the repository at this point in the history

  3. Simplify code and enhance docs

    @vladimirg-db
    vladimirg-db committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    1fa5050 View commit details
    Browse the repository at this point in the history

  4. Use stripMargin to prettyfy tests

    @vladimirg-db
    vladimirg-db committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    ba9f7cb View commit details
    Browse the repository at this point in the history