Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 0f7481e from #3100

…vided (#3100)

* fix: gracefully handle error when generative_qa_parameters is not provided

Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>

* fix: spotless apply

Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>

* docs: adding documentation link to error message

Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>

* tests: adding UT to test null params

Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>

---------

Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>
(cherry picked from commit 0f7481e)
@pyek-bot
Copy link
Collaborator

Please re-run tests. Failing due to flaky tests.

@ylwu-amzn ylwu-amzn merged commit 6779079 into 2.14 Dec 30, 2024
8 of 12 checks passed
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.

5 participants