Clean defensively within finally block

trinodb · Aug 27, 2019 · c975920 · c975920
1 parent 7727323
commit c975920
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/presto-product-tests/src/main/java/io/prestosql/tests/hive/TestRoles.java b/presto-product-tests/src/main/java/io/prestosql/tests/hive/TestRoles.java
@@ -289,7 +289,7 @@ public void testRevokeRoleFromOwner()
                             row("alice", "USER", "alice", "USER", "hive", "default", "test_table", "INSERT", "YES", null)));
         }
         finally {
-            onPrestoAlice().executeQuery("DROP TABLE hive.default.test_table");
+            onPrestoAlice().executeQuery("DROP TABLE IF EXISTS hive.default.test_table");
         }
     }
 
@@ -687,8 +687,8 @@ public void testAdminCanShowAllGrants()
                             row("alice", "USER", "bob", "USER", "hive", "default", "test_table_alice", "INSERT", "NO", null)));
         }
         finally {
-            onPrestoAlice().executeQuery("DROP TABLE hive.default.test_table_alice");
-            onPrestoAlice().executeQuery("DROP TABLE hive.default.test_table_bob");
+            onPrestoAlice().executeQuery("DROP TABLE IF EXISTS hive.default.test_table_alice");
+            onPrestoAlice().executeQuery("DROP TABLE IF EXISTS hive.default.test_table_bob");
             onPresto().executeQuery("REVOKE admin FROM alice");
         }
     }
@@ -728,9 +728,9 @@ public void testAdminCanShowGrantsOnlyFromCurrentSchema()
                             row("alice", "USER", "alice", "USER", "hive", "test", "test_table_bob", "INSERT", "YES", null)));
         }
         finally {
-            onPresto().executeQuery("DROP TABLE hive.default.test_table_bob");
-            onPrestoAlice().executeQuery("DROP TABLE hive.test.test_table_bob");
-            onPresto().executeQuery("DROP SCHEMA hive.test");
+            onPresto().executeQuery("DROP TABLE IF EXISTS hive.default.test_table_bob");
+            onPrestoAlice().executeQuery("DROP TABLE IF EXISTS hive.test.test_table_bob");
+            onPresto().executeQuery("DROP SCHEMA IF EXISTS hive.test");
             onPresto().executeQuery("REVOKE admin FROM alice");
         }
     }