PLUGIN-1823: Retrying all SQLTransientConnectionExceptions #23

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.

Sign up for GitHub

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

Jump to bottom

Open

sgarg-CS wants to merge 1 commit into develop from patch/plugin-1823

Collaborator

sgarg-CS commented May 6, 2025

Add Failsafe Retry poilcy to all the places in the DB Plugins where SQLTransientException or 'SQLTransientConnectionException` could be thrown.

Added three new properties (hidden from UI)

Initial Retry Duration (Default: 5sec)
Max Retry Duration (Default: 80 sec)
Max Retry Count (Default: 5)

sgarg-CS mentioned this pull request

PLUGIN-1823: Retry all SQLTransientConnectionException from DB sources/sinks. #22

Open

sgarg-CS requested review from psainics and vikasrathee-cs

May 6, 2025 14:47

psainics reviewed

View reviewed changes

database-commons/pom.xml Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/pom.xml Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/CommonSchemaReader.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/DBConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/DBRecord.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/DBRecord.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/DBRecord.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/RetryExceptions.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/action/AbstractDBArgumentSetter.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/RetryExceptions.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/RetryExceptions.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/RetryExceptions.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/RetryExceptions.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/action/AbstractDBArgumentSetter.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/action/AbstractQueryAction.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/action/AbstractDBAction.java Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/ConnectionConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/test/java/io/cdap/plugin/db/RetryPolicyUtilTest.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/util/RetryPolicyUtil.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/util/DBUtils.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/util/DBUtils.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/util/CloudSQLUtil.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

...ase-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBSpecificConnectorConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBConnectorConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/config/AbstractDBSpecificSourceConfig.java Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/ConnectionConfigAccessor.java Outdated Show resolved Hide resolved

sgarg-CS requested a review from psainics

May 15, 2025 11:21

psainics reviewed

View reviewed changes

pom.xml Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/ConnectionConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBConnectorConfig.java Show resolved Hide resolved

sgarg-CS requested a review from psainics

May 15, 2025 12:12

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/ConnectionConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBConnectorConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBConnectorConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBConnectorConfig.java Outdated Show resolved Hide resolved

sgarg-CS requested a review from psainics

May 16, 2025 05:14

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/config/AbstractDBSpecificSourceConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBConnectorConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBConnectorConfig.java Outdated Show resolved Hide resolved

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/connector/AbstractDBSpecificConnector.java Outdated Show resolved Hide resolved

sgarg-CS requested a review from psainics

May 16, 2025 07:29

psainics reviewed

View reviewed changes

database-commons/src/main/java/io/cdap/plugin/db/action/AbstractDBArgumentSetter.java Outdated Show resolved Hide resolved

psainics approved these changes

View reviewed changes

Collaborator

psainics left a comment

sgarg-CS force-pushed the patch/plugin-1823 branch from f920ad3 to 7fcb0a0 Compare

May 16, 2025 11:34


          PLUGIN-1823: Retrying all SQLTransientExceptions

9da75b8

psainics force-pushed the patch/plugin-1823 branch from 7fcb0a0 to 9da75b8 Compare

May 19, 2025 09:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet