guiyanakuang commented on a change in pull request #940:
URL: https://github.com/apache/orc/pull/940#discussion_r732552415
##########
File path: java/core/src/test/org/apache/orc/impl/TestEncryption.java
##########
@@ -140,4 +145,43 @@ public void testPushDownReadEncryption() throws
IOException {
read(true);
}
+ @Test
+ @SuppressWarnings("unchecked")
+ public void testConcurrentCreation() throws InterruptedException {
Review comment:
@dongjoon-hyun. After reading some guides, it is common to increase the
number of concurrency and test the number of repetitions of the method to
achieve the goal. But it still does not guarantee 100% failure. I agree merge
without test cases. 🤔
--
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: dev-unsubscr...@orc.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
