kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject [kudu] branch master updated: [master_sentry-itest] disable one test scenario
Date Wed, 15 May 2019 22:52:26 GMT
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
     new 96d6cc6  [master_sentry-itest] disable one test scenario
96d6cc6 is described below

commit 96d6cc6333083fdff0760a1c1c8571ab9b27ce78
Author: Alexey Serbin <alexey@apache.org>
AuthorDate: Tue May 14 21:13:54 2019 -0700

    [master_sentry-itest] disable one test scenario
    
    This is to fix a flakiness in SentryAuthzProviderCacheITest.CreateTables
    scenario: it 100% passes with the HEAD versions of Hive and Sentry but
    100% fails with Hive 2.1.1 and Sentry 2.1.0.  The scenario doesn't
    exhibit any other signs of flakiness.  Until the root cause is clear,
    let's keep this test scenario disabled.
    
    Change-Id: Ic397990e2fcda37ee6224a71a2ffe31aaa6414c6
    Reviewed-on: http://gerrit.cloudera.org:8080/13340
    Reviewed-by: Adar Dembo <adar@cloudera.com>
    Reviewed-by: Andrew Wong <awong@cloudera.com>
    Tested-by: Kudu Jenkins
---
 src/kudu/integration-tests/master_sentry-itest.cc | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/kudu/integration-tests/master_sentry-itest.cc b/src/kudu/integration-tests/master_sentry-itest.cc
index c42e91a..ddaf02d 100644
--- a/src/kudu/integration-tests/master_sentry-itest.cc
+++ b/src/kudu/integration-tests/master_sentry-itest.cc
@@ -880,7 +880,10 @@ TEST_F(SentryAuthzProviderCacheITest, ResetAuthzCacheConcurrentAlterTable)
{
 // misleading because CreateTable() reports a success to the client. The created
 // table indeed exists and is fully functional otherwise, but the corresponding
 // owner privilege record is absent in Sentry.
-TEST_F(SentryAuthzProviderCacheITest, CreateTables) {
+//
+// TODO(aserbin): clarify why it works with HEAD of the master branches
+//                of Sentry/Hive but fails with Sentry 2.1.0 and Hive 2.1.1.
+TEST_F(SentryAuthzProviderCacheITest, DISABLED_CreateTables) {
   constexpr const char* const kGhostTables[] = { "t10", "t11" };
 
   // Grant CREATE TABLE and METADATA privileges on the database.


Mime
View raw message