----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70516/#review214845 -----------------------------------------------------------
Ship it! Ship It! - Sarath Subramanian On April 23, 2019, 7:42 a.m., Nixon Rodrigues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70516/ > ----------------------------------------------------------- > > (Updated April 23, 2019, 7:42 a.m.) > > > Review request for atlas, Ashutosh Mestry, Madhan Neethiraj, and Sarath > Subramanian. > > > Bugs: ATLAS-3129 > https://issues.apache.org/jira/browse/ATLAS-3129 > > > Repository: atlas > > > Description > ------- > > AtlasBaseClient uses Hadoop's SSLFactory to create the > ReloadingX509TrustManager > > https://github.com/apache/atlas/blob/b919d3d0964efbe1eda40724545d5fa506100a02/client/common/src/main/java/org/apache/atlas/security/SecureClientUtils.java#L143 > > currently, there is no way to stop/destroy method of the > ReloadingX509TrustManager, this creates "Truststore reloader thread" leak in > hiveserver2 if the user is running any HOOK/UDF which uses the AtlasClient. > > Solution:- Refactoring of code and added destroy method to destroy SSLFactory > instance. > > > Diffs > ----- > > > addons/hbase-bridge/src/main/java/org/apache/atlas/hbase/bridge/HBaseBridge.java > 17d617d19 > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/HiveMetaStoreBridge.java > 21ceda200 > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java > 214dd2512 > client/common/src/main/java/org/apache/atlas/AtlasBaseClient.java cb7340fb6 > > client/common/src/main/java/org/apache/atlas/security/SecureClientUtils.java > aeae300a6 > webapp/src/main/java/org/apache/atlas/examples/QuickStart.java c043b65eb > > > Diff: https://reviews.apache.org/r/70516/diff/2/ > > > Testing > ------- > > In progress > > > Thanks, > > Nixon Rodrigues > >
