Skip to content

fix: AmazonBedrockRetriever was missing some last details - #3842

Merged
davidsbatista merged 3 commits into
mainfrom
fix/AmazonBedrockRetriever
Aug 26, 2026
Merged

davidsbatista merged 3 commits into
mainfrom
fix/AmazonBedrockRetriever

Conversation

@davidsbatista

Copy link
Copy Markdown
Contributor

Related Issues

Checklist

@davidsbatista

Copy link
Copy Markdown
Contributor Author

integrations page PR is here: deepset-ai/haystack-integrations#578

@github-actions

Copy link
Copy Markdown
Contributor

Coverage report (amazon_bedrock)

This PR does not seem to contain any modification to coverable code.

@davidsbatista
davidsbatista marked this pull request as ready for review August 24, 2026 13:14
@davidsbatista
davidsbatista requested a review from a team as a code owner August 24, 2026 13:14
@davidsbatista
davidsbatista requested review from anakin87 and bogdankostic and removed request for a team and bogdankostic August 24, 2026 13:14

@bogdankostic bogdankostic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Two points from the issue are still missing though:

@davidsbatista

Copy link
Copy Markdown
Contributor Author

Looks good! Two points from the issue are still missing though:

  • changing the name to AmazonBedrockKnowledgeBaseRetriever

That was the original name, and @anakin87 suggested changing it to BedrockKnowledgeBaseRetriever

@bogdankostic

Copy link
Copy Markdown
Contributor

Looks good! Two points from the issue are still missing though:

  • changing the name to AmazonBedrockKnowledgeBaseRetriever

That was the original name, and @anakin87 suggested changing it to BedrockKnowledgeBaseRetriever

Hmm, IMO AmazonBedrockKnowledgeBaseRetriever would be more consistent with the naming of the other Amazon Bedrock components, but fine, let's go with BedrockKnowledgeBaseRetriever.

@bogdankostic bogdankostic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

@davidsbatista
davidsbatista merged commit 471d98c into main Aug 26, 2026
17 checks passed
@davidsbatista
davidsbatista deleted the fix/AmazonBedrockRetriever branch August 26, 2026 12:24
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.

2 participants