-
Notifications
You must be signed in to change notification settings - Fork 2.4k
added cluster and index setting to control plan explain #20222
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
added cluster and index setting to control plan explain #20222
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
| if is_query_plan_explain_enabled { | ||
| println!("---- Explain plan ----"); | ||
| let clone_df = dataframe.clone().explain(false, true); | ||
| clone_df?.show().await?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove ? and handle exceptions
|
❌ Gradle check result for 301c9cb: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
50bf1d1 to
b5ac70c
Compare
|
❌ Gradle check result for b5ac70c: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
b5ac70c to
3e43a17
Compare
|
❌ Gradle check result for 3e43a17: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
3e43a17 to
92dd9dd
Compare
|
❌ Gradle check result for 92dd9dd: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
92dd9dd to
50945a5
Compare
|
❌ Gradle check result for 50945a5: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
32ab218
into
opensearch-project:feature/datafusion
Description
Added index setting
index.search.query_plan.explainto print query plan explain.Default is false.
Added cluster setting
search.query_plan.explainto print query plan explain.Default is false.
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.