Skip to content

Add cluster command for PPL#5265

Draft
ritvibhatt wants to merge 1 commit intoopensearch-project:mainfrom
ritvibhatt:ppl-cluster-command
Draft

Add cluster command for PPL#5265
ritvibhatt wants to merge 1 commit intoopensearch-project:mainfrom
ritvibhatt:ppl-cluster-command

Conversation

@ritvibhatt
Copy link
Contributor

Description

[Describe what this change achieves]

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • New functionality has javadoc added.
  • New functionality has a user manual doc added.
  • New PPL command checklist all confirmed.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff or -s.
  • Public documentation issue/PR created.

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.

Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
@github-actions
Copy link
Contributor

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit 8f2804e.

PathLineSeverityDescription
core/src/main/java/org/opensearch/sql/calcite/udf/udaf/ClusterLabelAggFunction.java172lowSystem.err.println used for error output instead of a logging framework. While not malicious, this could leak internal cluster IDs and exception messages to stderr in production environments and represents an anomaly compared to the rest of the codebase.

The table above displays the top 10 most important findings.

Total: 1 | Critical: 0 | High: 0 | Medium: 0 | Low: 1


Pull Requests Author(s): Please update your Pull Request according to the report above.

Repository Maintainer(s): You can bypass diff analyzer by adding label skip-diff-analyzer after reviewing the changes carefully, then re-run failed actions. To re-enable the analyzer, remove the label, then re-run all actions.


⚠️ Note: The Code-Diff-Analyzer helps protect against potentially harmful code patterns. Please ensure you have thoroughly reviewed the changes beforehand.

Thanks.

@github-actions
Copy link
Contributor

Failed to generate code suggestions for PR

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.

1 participant