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

[Backport 2.x] Add batch_size param for text_embedding processor #1312

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 6c3e68f from #1298.

* Add batchSize parameter for text_embedding processor

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

* throw IllegalArgumentException when batchSize is not a positive integer

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

* test: add test cases for BatchSize param

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

* test: exception when batchSize is zero or negative integer

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

* refactor: use assertNotNull for readability & convention

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

* update CHANGELOG about #1298 PR

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

* apply code convention to keep the codes spotless

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

---------

Signed-off-by: YeonghyeonKO <[email protected]>
Signed-off-by: Thomas Farr <[email protected]>
Co-authored-by: Thomas Farr <[email protected]>
(cherry picked from commit 6c3e68f)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@Xtansia Xtansia merged commit 99d0653 into 2.x Nov 18, 2024
99 of 100 checks passed
@github-actions github-actions bot deleted the backport/backport-1298-to-2.x branch November 18, 2024 21:09
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