[GitHub] flink pull request #6355: [FLINK-9878][network][ssl] add more low-level ssl ...

[pnowojski]

Github user pnowojski commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6355#discussion_r204300332
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/NettyClient.java
 ---
    @@ -52,6 +56,9 @@
     
        private Bootstrap bootstrap;
     
    +   @Nullable
    --- End diff --
    
    Same argument as somewhere else: `Optional`.  You mark `clientSSLConfig` as 
nullable and never check it for not null. 


---