AMashenkov commented on code in PR #3170:
URL: https://github.com/apache/ignite-3/pull/3170#discussion_r1494673171


##########
modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItCreateTableDdlTest.java:
##########
@@ -340,24 +335,4 @@ public void 
testItIsNotPossibleToCreateTablesInSystemSchema(String schema) {
     private static Stream<Arguments> reservedSchemaNames() {
         return SYSTEM_SCHEMAS.stream().map(Arguments::of);
     }
-
-    @Disabled("https://issues.apache.org/jira/browse/IGNITE-20680";)
-    @Test
-    public void concurrentDrop() {

Review Comment:
   This test checks a race condition, when storages has been destroyed 
concurrently.
   Actually, the test itself has a race.
   It look safe to drop it, as there are/will be other tests
   * table and table storages doesn't destroyed on DROP, but on DESTROY event.
   * DESTROY event called explicitly. (Later will be triggered on LWM change)
   * LWM waits for all running transactions
   * a transaction can NOT be started with timestamp before LWM
   * 



-- 
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]

Reply via email to