Skip to content

Conversation

@twosom
Copy link
Contributor

@twosom twosom commented Dec 26, 2025

Linked issue: #385, #321

Purpose of change

Added documentation for ElasticsearchVectorStore.

Tests

API

Documentation

  • doc-needed
  • doc-not-needed

@github-actions github-actions bot added priority/major Default priority of the PR or issue. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. doc-label-missing The Bot applies this label either because none or multiple labels were provided. labels Dec 26, 2025
@github-actions
Copy link

@twosom Please add the following content to your PR description and select a checkbox:

- [ ] `doc-needed` 
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->

@twosom twosom force-pushed the doc-elasticsearch-vectorstore branch from 3e293e7 to 27bd922 Compare December 26, 2025 03:19
@github-actions github-actions bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing The Bot applies this label either because none or multiple labels were provided. labels Dec 26, 2025
@twosom
Copy link
Contributor Author

twosom commented Jan 5, 2026

@xintongsong

Thank you for the review.
Sorry for the late response. I had some personal matters to attend to.

I will update the PR today.

@twosom twosom force-pushed the doc-elasticsearch-vectorstore branch from 82ad3d6 to 81d31f1 Compare January 5, 2026 03:20
Copy link
Contributor

@xintongsong xintongsong left a comment

Choose a reason for hiding this comment

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

Thanks for addressing my comments, @twosom. The PR looks quite nice. I just have a few minor comments. Please take a look.

Copy link
Contributor

@xintongsong xintongsong left a comment

Choose a reason for hiding this comment

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

Thanks for addressing my comments. LGTM

@xintongsong xintongsong merged commit 71b8bff into apache:main Jan 12, 2026
18 checks passed
@xintongsong xintongsong added doc-included Your PR already contains the necessary documentation updates. and removed doc-not-needed Your PR changes do not impact docs labels Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-included Your PR already contains the necessary documentation updates. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants