Skip to content
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

error handling changes for Redshift #2275

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Trianz-Akshay
Copy link
Contributor

@Trianz-Akshay Trianz-Akshay commented Sep 19, 2024

Issue #, if available:

Description of changes:
Error handling improvement changes for Athena Redshift connector. As Redshift extended Postgres so updated the file in Postgres connector.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Trianz-Akshay Trianz-Akshay changed the title error handling changes for postgressql error handling changes for Redshift Sep 19, 2024
@Trianz-Akshay
Copy link
Contributor Author

Access Denied Error Handling redshift.docx
PFA docs for access denied changes.

@aaronsven
Copy link

So access denied is also when the user has an invalid password. Are we covering that case?

@Trianz-Akshay
Copy link
Contributor Author

So access denied is also when the user has an invalid password. Are we covering that case?

Invalid passwords are handled under InvalidCredentialsException, which covers scenarios like access denied due to incorrect credentials.
This is addressed in the JDBC module and the SDK as Redshift and Snowflake are DBC connectors, ensuring that such cases are appropriately managed.
I am attaching a document with screenshots for the same.
Invalid Credentials Error Handling Redshift.docx
Invalid credentials Error Handling Snowflake.docx

Link to athena-jdbc PR. (#2276)
Link to athena-federation-sdk PR. (#2278)

@ritiktrianz ritiktrianz force-pushed the feature/error_handling_postgres branch from c23b035 to 0e9880b Compare November 6, 2024 07:51
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.

4 participants