roczei opened a new pull request, #523: URL: https://github.com/apache/incubator-livy/pull/523
## What changes were proposed in this pull request? Introduced an optional TLS encryption for the Curator based ZooKeeper client used during session recovery. Changes by Asif Khatri: - Added five LivyConf entries (LIVY_ZK_CLIENT_SECURE, LIVY_ZK_CLIENT_SOCKET, LIVY_ZK_KEYSTORE_PASS, LIVY_ZK_TRUSTSTORE_FILE, LIVY_ZK_TRUSTSTORE_PASS) and document them in livy.conf.template. - Added ZooKeeperManager.createZKClientConfig that assembles a ZKClientConfig from keystore/truststore settings; applied to the CuratorFramework builder only when livy.server.zk.client.secure=true. ## How was this patch tested? Test improvements: - Extracted mockCurator() helper in ZooKeeperStateStoreSpec to eliminate repeated CuratorFramework mock setup boilerplate. - Added an "SSL config" describe block with withSslMock helper (SslTestFixture) and tests covering SSL property mapping and both the secure and non-secure construction paths. ## Was this patch authored or co-authored using generative AI tooling? Yes, this was co-authored using Cursor to help generate the new test cases. Generated-by: Cursor 3.5.17 (claude-4.6-sonnet-medium-thinking) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
