andreachild commented on code in PR #2753:
URL: https://github.com/apache/tinkerpop/pull/2753#discussion_r1751032525
##########
gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/Channelizer.java:
##########
@@ -87,6 +88,7 @@ default String getScheme(final boolean sslEnabled) {
abstract class AbstractChannelizer extends
ChannelInitializer<SocketChannel> implements Channelizer {
protected Connection connection;
protected Cluster cluster;
+ protected SslHandler sslHandler;
Review Comment:
There should not be multiple threads setting the SslHandler as it is
initialized as part of the Connection constructor which calls `new
Channelizer.HttpChannelizer()` and eventually `ChannelInitializer.initChannel`
which only executes `initChannel(Channel)` once per `ChannelHandlerContext`.
--
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]
