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

    https://github.com/apache/cassandra/pull/239#discussion_r204458523
  
    --- Diff: src/java/org/apache/cassandra/config/DatabaseDescriptor.java ---
    @@ -2260,6 +2260,20 @@ public static int getStreamingConnectionsPerHost()
             return conf.streaming_connections_per_host;
         }
     
    +    public static boolean isFullSSTableTransfersEnabled()
    +    {
    +        if (conf.server_encryption_options.enabled || 
conf.server_encryption_options.optional)
    +        {
    +            logger.debug("Internode encryption enabled. Disabling zero 
copy SSTable transfers for streaming.");
    --- End diff --
    
    Nobody will ever see this at `debug` level. We should at minimum `warn` if 
`streaming_zerocopy_sstables_enabled` is and internode encryption are both 
enabled at the same time.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to