aws / aws-sdk-pandas

pandas on AWS - Easy integration with Athena, Glue, Redshift, Timestream, Neptune, OpenSearch, QuickSight, Chime, CloudWatchLogs, DynamoDB, EMR, SecretManager, PostgreSQL, MySQL, SQLServer and S3 (Parquet, CSV, JSON and EXCEL).

https://aws-sdk-pandas.readthedocs.io

Apache License 2.0

3.94k stars 701 forks source link

fix: OpenSearch import errors #2939

Closed LeonLuttenberger closed 3 months ago

LeonLuttenberger commented 3 months ago

Detail

If a customer has opensearchpy installed but not jsonpath_ng, the import of awswrangler will fail. This goes against our current protocol, which is that we throw an appropriate import error when the customer tries to invoke a method which requires an optional dependency.

The jsonpath_ng dependency is also only required for certain cases when loading JSON data, so I'm only appending the annotation to the _get_documents_w_json_path function.

Issue

2938

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

malachi-constant commented 3 months ago

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: f386193de40baa565e5de463a7d34544c816f546
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

malachi-constant commented 3 months ago

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: f386193de40baa565e5de463a7d34544c816f546
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository