You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by st...@apache.org on 2021/09/16 05:41:03 UTC

[phoenix] branch master updated: PHOENIX-6547 BasePermissionsIT is still a bit flakey

This is an automated email from the ASF dual-hosted git repository.

stoty pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/master by this push:
     new e631d24  PHOENIX-6547 BasePermissionsIT is still a bit flakey
e631d24 is described below

commit e631d24bcae16ad8c71374fa1d3f42c01c762315
Author: Istvan Toth <st...@apache.org>
AuthorDate: Wed Sep 15 15:56:20 2021 +0200

    PHOENIX-6547 BasePermissionsIT is still a bit flakey
---
 .../it/java/org/apache/phoenix/end2end/BasePermissionsIT.java  | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/phoenix-core/src/it/java/org/apache/phoenix/end2end/BasePermissionsIT.java b/phoenix-core/src/it/java/org/apache/phoenix/end2end/BasePermissionsIT.java
index 7ee5aea..9a87544 100644
--- a/phoenix-core/src/it/java/org/apache/phoenix/end2end/BasePermissionsIT.java
+++ b/phoenix-core/src/it/java/org/apache/phoenix/end2end/BasePermissionsIT.java
@@ -1391,7 +1391,10 @@ public abstract class BasePermissionsIT extends BaseTest {
             // this user
             verifyAllowed(readTable(phoenixTableName, indexName1), unprivilegedUser);
             verifyAllowed(readTable(phoenixTableName), regularUser1);
-            verifyAllowed(rebuildIndex(indexName1, phoenixTableName), regularUser1);
+            retryVerifyOperation(() -> {
+                verifyAllowed(rebuildIndex(indexName1, phoenixTableName), regularUser1);
+                return null;
+            }, UndeclaredThrowableException.class, 4);
             verifyAllowed(addColumn(phoenixTableName, "val1"), regularUser1);
             verifyAllowed(addProperties(phoenixTableName, "GUIDE_POSTS_WIDTH", "100"), regularUser1);
             verifyAllowed(dropView(viewName1), regularUser1);
@@ -1405,7 +1408,10 @@ public abstract class BasePermissionsIT extends BaseTest {
 
             // check again with super users
             verifyAllowed(createTable(phoenixTableName), superUser2);
-            verifyAllowed(createIndex(indexName1, phoenixTableName), superUser2);
+            retryVerifyOperation(() -> {
+                verifyAllowed(createIndex(indexName1, phoenixTableName), superUser2);
+                return null;
+            }, UndeclaredThrowableException.class, 4);
             verifyAllowed(createView(viewName1, phoenixTableName), superUser2);
             verifyAllowed(readTable(phoenixTableName), superUser2);
             verifyAllowed(dropView(viewName1), superUser2);