mridulm commented on code in PR #43387:
URL: https://github.com/apache/spark/pull/43387#discussion_r1369598438
##########
core/src/test/scala/org/apache/spark/network/netty/NettyBlockTransferSecuritySuite.scala:
##########
@@ -112,6 +179,19 @@ class NettyBlockTransferSecuritySuite extends
SparkFunSuite with MockitoSugar wi
}
}
+ test("security with aes encryption over ssl") {
Review Comment:
~Agree, it is not a case we fail over. Let us keep it.~
We return `false` in case both are enabled - so even though users can
specify, it wont work and there is no expectation for it to work (in case this
changes in future).
We can add a boolean to control which tests to disable in SSL and enable it
in the subclass
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
