Fix SSL connections (broken by THRIFT-5595)

cloudera / impyla

Python DB API 2.0 client for Impala and Hive (HiveServer2 protocol)

Apache License 2.0

731 stars 248 forks source link

Fix SSL connections (broken by THRIFT-5595) #496

Closed csringhofer closed 2 years ago

MikaelSmith commented 2 years ago

Any way we could add a test around this?

csringhofer commented 2 years ago

Added a commit with some basic tests. Authentication tests are not completely automated in Impyla, as first an Impala cluster has to be set up.

I was thinking about how to automate these better - Impyla auth tests could be added to the Impala test suite, where running of custom clusters is convenient, or Impyla tests could be extended to restart the Impala cluster when needed. For the current fix I wouldn't dive into this deeper.