Skip to content

Conversation

sobychacko
Copy link
Contributor

Bedrock does not support topK and setting it via BedrockConverseProxyChatProperties does not have any effect.

Fixes: spring-projects#3621

Bedrock does not support topK and setting it via BedrockConverseProxyChatProperties
does not have any effect.

Signed-off-by: Soby Chacko <[email protected]>
@sobychacko sobychacko force-pushed the remove-bedrock-topK-warning branch from efde597 to 6e44f73 Compare June 19, 2025 23:23
@ilayaperumalg ilayaperumalg merged commit 8c411ab into spring-projects:main Jun 20, 2025
2 checks passed
@ilayaperumalg ilayaperumalg self-assigned this Jun 20, 2025
spring-builds pushed a commit that referenced this pull request Jun 20, 2025
Fixes: #3621

Bedrock does not support topK and setting it via BedrockConverseProxyChatProperties
does not have any effect.

Signed-off-by: Soby Chacko <[email protected]>
(cherry picked from commit 8c411ab)
scionaltera pushed a commit to scionaltera/spring-ai that referenced this pull request Sep 3, 2025
…rojects#3621)

Fixes: spring-projects#3621

Auto-cherry-pick to 1.0.x

Bedrock does not support topK and setting it via BedrockConverseProxyChatProperties
does not have any effect.

Signed-off-by: Soby Chacko <[email protected]>
chedim pushed a commit to couchbaselabs/spring-ai that referenced this pull request Sep 19, 2025
…rojects#3621)

Fixes: spring-projects#3621

Auto-cherry-pick to 1.0.x

Bedrock does not support topK and setting it via BedrockConverseProxyChatProperties
does not have any effect.

Signed-off-by: Soby Chacko <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants