Skip to content

Conversation

@faviansamatha
Copy link
Contributor

@faviansamatha faviansamatha commented Dec 5, 2025

Description

This PR adds a backend mysql client using the Tortoise ORM support.

  • Adds async connection pool with health checks
  • Adds AwsMySQL client
  • Adds async wrapper utility to be compatible with tortoise
  • Added integration tests for tortoise
  • Added unit tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@faviansamatha faviansamatha force-pushed the feat/tortoise branch 11 times, most recently from a7d1332 to 75164e6 Compare December 6, 2025 01:27
@faviansamatha faviansamatha force-pushed the feat/tortoise branch 3 times, most recently from 61cf1df to 745b003 Compare December 18, 2025 01:36
@faviansamatha faviansamatha force-pushed the feat/tortoise branch 3 times, most recently from 82e82d1 to 80ffac9 Compare December 18, 2025 05:21
is_cached_token = (token_info is not None and not token_info.is_expired())
if not self._plugin_service.is_login_exception(error=e) or not is_cached_token:
raise AwsWrapperError(Messages.get_formatted("IamAuthPlugin.ConnectException", e)) from e
raise
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need to remove the wrapped error. If we get a failover and we get a network error, the failover plugin will not detect it.

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