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.

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

[Enhancement] (nereids)implement showCreateCatalogCommand in nereids #43068

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

msridhar78
Copy link
Contributor

@msridhar78 msridhar78 commented Nov 1, 2024

Issue Number: close #42740

implement showCreateCatalogCommand in nereids

Regression test result:

Changed Session Variables:

VarName CurrentValue DefaultValue
character_set_results NULL utf8mb4
enable_parallel_result_sink true false
sql_mode STRICT_TRANS_TABLES
variable_version 0 300
experimental_enable_nereids_distribute_planner false true

2024-11-19 15:27:31.633 INFO [suite-thread-1] (Suite.groovy:1316) - Execute tag: cmd, sql: show create catalog es
2024-11-19 15:27:31.641 INFO [suite-thread-1] (SuiteContext.groovy:392) - Delete /root/doris/doris/regression-test/data/nereids_p0/show/test_show_create_catalog.out
2024-11-19 15:27:31.642 INFO [suite-thread-1] (SuiteContext.groovy:394) - Generate /root/doris/doris/regression-test/data/nereids_p0/show/test_show_create_catalog.out
2024-11-19 15:27:31.665 INFO [suite-thread-1] (Suite.groovy:387) - Execute sql: drop catalog if exists es
2024-11-19 15:27:31.675 INFO [suite-thread-1] (ScriptContext.groovy:120) - Run test_show_create_catalog in /root/doris/doris/regression-test/suites/nereids_p0/show/test_show_create_catalog.groovy succeed
2024-11-19 15:27:52.891 INFO [main] (RegressionTest.groovy:281) - Start to run single scripts
2024-11-19 15:29:23.320 INFO [main] (RegressionTest.groovy:410) - Success suites:
/root/doris/doris/regression-test/suites/nereids_p0/show/test_show_create_catalog.groovy: group=p0,external,hive,external_docker,external_docker_hive, name=test_show_create_catalog
2024-11-19 15:29:23.321 INFO [main] (RegressionTest.groovy:492) - All suites success.

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@msridhar78
Copy link
Contributor Author

run buildall

@msridhar78 msridhar78 force-pushed the show_create_catalog1 branch from 11dfb0d to f28e346 Compare November 2, 2024 06:04
@msridhar78 msridhar78 marked this pull request as draft November 4, 2024 04:39
@msridhar78 msridhar78 force-pushed the show_create_catalog1 branch from f28e346 to a9fd178 Compare November 4, 2024 15:32
@msridhar78 msridhar78 marked this pull request as ready for review November 4, 2024 15:32
/**
* Represents the command for SHOW CREATE CATALOG.
*/
public class ShowCreateCatalogCommand extends Command implements NoForward {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
public class ShowCreateCatalogCommand extends Command implements NoForward {
public class ShowCreateCatalogCommand extends ShowCommand {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done - pls review

"hosts"="http://127.0.0.1:9200"
);"""

result = sql """show create catalog ${catalog_name}"""
Copy link
Contributor

Choose a reason for hiding this comment

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

checkNereidsExecute("""show create catalog ${catalog_name}""")

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done - please review

@msridhar78
Copy link
Contributor Author

run buildall

Copy link
Contributor

@starocean999 starocean999 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Nov 20, 2024
Copy link
Contributor

PR approved by at least one committer and no changes requested.

Copy link
Contributor

PR approved by anyone and no changes requested.

@starocean999 starocean999 merged commit 6eafef3 into apache:master Nov 20, 2024
28 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by one committer. reviewed
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Enhancement] (nereids)implement showCreateCatalogCommand in nereids
4 participants