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 Elasticsearch connector #2279

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

Conversation

Trianz-Akshay
Copy link
Contributor

Issue #, if available:

Description of changes:
Error handling improvement changes for Athena Elasticsearch 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
Copy link
Contributor Author

I've pushed the changes addressing the access denied exception case. Please find the attached document for further details.
Uploading Access- Denied Error Handling OpenSearch.docx…

@Trianz-Akshay Trianz-Akshay changed the title error handling changes for elasticsearch Error handling changes for Elasticsearch connector Oct 21, 2024
@ritiktrianz ritiktrianz force-pushed the feature/error_handling_athena_elasticsearch branch from 38a1fe9 to fc158e6 Compare November 7, 2024 08:52
# Conflicts:
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/AWSRequestSigningApacheInterceptor.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/AwsRestHighLevelClient.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/ElasticsearchDomainMapProvider.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/ElasticsearchMetadataHandler.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/ElasticsearchRecordHandler.java
# Conflicts:
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/AWSRequestSigningApacheInterceptor.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/AwsRestHighLevelClient.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/ElasticsearchDomainMapProvider.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/ElasticsearchMetadataHandler.java
#	athena-elasticsearch/src/main/java/com/amazonaws/athena/connectors/elasticsearch/ElasticsearchRecordHandler.java
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.

2 participants