István Fajth created HADOOP-18977: ------------------------------------- Summary: Create curator client the same way possibly with the same code for all services Key: HADOOP-18977 URL: https://issues.apache.org/jira/browse/HADOOP-18977 Project: Hadoop Common Issue Type: Improvement Reporter: István Fajth
Currently ZKSignerSecretProvider and ZKDelegationTokenSecretManager classes are using the same ZookeeperClient class to create their respective Zookeeper connection via a Curator client. In order to have a standardized way of creating Zookeeper connections, we have two areas to fix: - YARN and its Curator instantiation - HDFS ZKFC, which currently uses a Zookeeper client directly, so it should also be migrated to Curator if we want to utilize the common instantiation. The problem that we would solve is that currently we have 3 places where we create ZK connections, and all of them has its own things, along with some duplications. We also have the HadoopZookeeperFactory, that I think we do not need. The only thing that is different for this is how we set up SASL permission provider for the connection. I believe that can also be unified, with that the whole Zookeeper connection creation can be pushed down to the recently added code that handles this for ZKDelegationTokenSecretManager and ZKSignerSecretProvider. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-dev-h...@hadoop.apache.org