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
Add support for Glue endpoint URL #3239
Conversation
Tested manually, but it requires -Dcom.amazonaws.sdk.disableCertChecking=true, due to misalignment between cert name (glue..amazonaws.com) and actual hostname used in endpoint. |
0e04777
to
ec55e21
Compare
Fixed by providing SSL Factory with NoopHostnameVerifier. |
Please reabase instead of merge commit |
presto-hive/src/main/java/io/prestosql/plugin/hive/metastore/glue/GlueHiveMetastore.java
Outdated
Show resolved
Hide resolved
ec55e21
to
8b68280
Compare
8b68280
to
45787a9
Compare
presto-hive/src/main/java/io/prestosql/plugin/hive/metastore/glue/GlueHiveMetastore.java
Outdated
Show resolved
Hide resolved
45787a9
to
7c6c7bf
Compare
7c6c7bf
to
aef53ff
Compare
Cherry pick of trinodb/trino#3239 Co-authored-by: Pawel Palucha <pawel.palucha@starburstdata.com> Co-authored-by: Ashhar Hasan <hashhar_dev@outlook.com>
Cherry pick of trinodb/trino#3239 Co-authored-by: Pawel Palucha <pawel.palucha@starburstdata.com> Co-authored-by: Ashhar Hasan <hashhar_dev@outlook.com>
No description provided.