Skip to content

Fix indentation in _conversion_stream_with_retry function#21354

Open
AmitabhDey-byte wants to merge 1 commit intorun-llama:mainfrom
AmitabhDey-byte:llama_index/llms/bedrock_converse/utils.py
Open

Fix indentation in _conversion_stream_with_retry function#21354
AmitabhDey-byte wants to merge 1 commit intorun-llama:mainfrom
AmitabhDey-byte:llama_index/llms/bedrock_converse/utils.py

Conversation

@AmitabhDey-byte
Copy link
Copy Markdown

@AmitabhDey-byte AmitabhDey-byte commented Apr 9, 2026

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #

  • Introduced an internal _connect coroutine wrapped with @retry_decorator
  • Applied retry only to the client.converse_stream() call
  • Left streaming iteration outside retry, since retrying a partially consumed stream is not meaningful

New Package?

no new package added

Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

  • No

Version Bump?

Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

  • No

Type of Change

Please delete options that are not relevant.

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

  • I believe this change is already covered by existing unit tests

Suggested Checklist:

  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant