Skip to content

Update sql docs for querying multiple indices#5340

Merged
ahkcs merged 2 commits into
opensearch-project:mainfrom
ritvibhatt:update-multi-index-docs
Apr 13, 2026
Merged

Update sql docs for querying multiple indices#5340
ahkcs merged 2 commits into
opensearch-project:mainfrom
ritvibhatt:update-multi-index-docs

Conversation

@ritvibhatt

@ritvibhatt ritvibhatt commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Description

  • Updates SQL docs to clarify that when querying mulitple indices, the list needs to be enclosed in backticks

Related Issues

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.

@ritvibhatt
update sql docs for querying multiple indexes
ebf45d3 
Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
@ritvibhatt
fix wording
cb69526 
Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
@github-actions

github-actions Bot commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@dai-chen dai-chen added the documentation Improvements or additions to documentation label Apr 13, 2026
@ahkcs ahkcs merged commit d48aee2 into opensearch-project:main Apr 13, 2026
38 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dai-chen dai-chen dai-chen approved these changes

@ahkcs ahkcs ahkcs approved these changes

@ps48 ps48 Awaiting requested review from ps48 ps48 is a code owner

@joshuali925 joshuali925 Awaiting requested review from joshuali925 joshuali925 is a code owner

@mengweieric mengweieric Awaiting requested review from mengweieric mengweieric is a code owner

@vamsimanohar vamsimanohar Awaiting requested review from vamsimanohar vamsimanohar is a code owner

@Swiddis Swiddis Awaiting requested review from Swiddis Swiddis is a code owner

@penghuo penghuo Awaiting requested review from penghuo penghuo is a code owner

@anirudha anirudha Awaiting requested review from anirudha anirudha is a code owner

@acarbonetto acarbonetto Awaiting requested review from acarbonetto acarbonetto is a code owner

@ykmr1224 ykmr1224 Awaiting requested review from ykmr1224 ykmr1224 is a code owner

@LantaoJin LantaoJin Awaiting requested review from LantaoJin LantaoJin is a code owner

@noCharger noCharger Awaiting requested review from noCharger noCharger is a code owner

@qianheng-aws qianheng-aws Awaiting requested review from qianheng-aws qianheng-aws is a code owner

@yuancu yuancu Awaiting requested review from yuancu yuancu is a code owner

@RyanL1997 RyanL1997 Awaiting requested review from RyanL1997 RyanL1997 is a code owner

@songkant-aws songkant-aws Awaiting requested review from songkant-aws songkant-aws is a code owner

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ritvibhatt @dai-chen @ahkcs