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

Update TextEmbeddingProcessor.java #1127

Merged
merged 3 commits into from
Sep 30, 2024
Merged

Conversation

ehc0825
Copy link
Contributor

@ehc0825 ehc0825 commented Aug 6, 2024

change type

Description

Describe what this change achieves.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

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.

change type

Signed-off-by: leeeeeeeeeeehc <[email protected]>
@dblock
Copy link
Member

dblock commented Aug 6, 2024

Thanks, can you please explain why?

@Xtansia
Copy link
Collaborator

Xtansia commented Aug 27, 2024

@ehc0825 Could you please add an entry to the ### Fixed section under ## [Unreleased 2.x] of the CHANGELOG?

Signed-off-by: Thomas Farr <[email protected]>
Xtansia
Xtansia previously approved these changes Sep 30, 2024
@Xtansia Xtansia merged commit 28e8b89 into opensearch-project:main Sep 30, 2024
54 of 56 checks passed
@Xtansia Xtansia added the backport 2.x Backport to 2.x branch label Sep 30, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 30, 2024
* Update TextEmbeddingProcessor.java

change type

Signed-off-by: leeeeeeeeeeehc <[email protected]>

* Add changelog entry

Signed-off-by: Thomas Farr <[email protected]>

---------

Signed-off-by: leeeeeeeeeeehc <[email protected]>
Signed-off-by: Thomas Farr <[email protected]>
Co-authored-by: Thomas Farr <[email protected]>
(cherry picked from commit 28e8b89)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Xtansia added a commit that referenced this pull request Sep 30, 2024
* Update TextEmbeddingProcessor.java

change type



* Add changelog entry



---------




(cherry picked from commit 28e8b89)

Signed-off-by: leeeeeeeeeeehc <[email protected]>
Signed-off-by: Thomas Farr <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Thomas Farr <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants