Skip to content

[🍒][Plugin-1779] Add TRANSACTION_ISOLATION_LEVEL config in MySQL, PostgreSQL & SQL Server plugins #589

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

sgarg-CS
Copy link
Contributor

@sgarg-CS sgarg-CS commented May 5, 2025

[🍒]

🍒 [cherrypick]

Commits :

PR:


Add TRANSACTION_ISOLATION_LEVEL config in MySQL, PostgreSQL & SQL Server plugins

JIRA : Plugin-1779

Description

This PR adds the TRANSACTION_ISOLATION_LEVEL configuration property in MySQL, PostgreSQL & SQL Server plugins.

Added support for specifying the transaction isolation level in database plugins by introducing a new optional property "transactionIsolationLevel" in the AbstractDBSpecificConnectorConfig class — the parent class for MySQL, PostgreSQL, SQL Server and Oracle connector configurations.

Preview

image

Current branch sanity on CDAP (6.10.1)

image

@psainics psainics added the build label May 5, 2025
@psainics psainics requested a review from ritwiksahani May 5, 2025 11:43
Copy link

@ritwiksahani ritwiksahani left a comment

Choose a reason for hiding this comment

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

Please pull latest changes from the release/1.11 branch it is showing that the branch is not in sync.

@sgarg-CS sgarg-CS force-pushed the cherry-pick/f4650e13274f73e8c173c2895170f512ef4245ee branch from 472c4e9 to ede853b Compare May 6, 2025 06:23
…lugins

Add transaction isolation level to PostgreSQL, MYSQL and MSSQL Plugins.
@sgarg-CS sgarg-CS force-pushed the cherry-pick/f4650e13274f73e8c173c2895170f512ef4245ee branch from ede853b to 7959157 Compare May 6, 2025 06:25
@sgarg-CS
Copy link
Contributor Author

sgarg-CS commented May 6, 2025

Please pull latest changes from the release/1.11 branch it is showing that the branch is not in sync.

Synced

@sgarg-CS sgarg-CS merged commit 891ff53 into data-integrations:release/1.11 May 6, 2025
5 of 10 checks passed
@sgarg-CS sgarg-CS deleted the cherry-pick/f4650e13274f73e8c173c2895170f512ef4245ee branch May 6, 2025 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants