You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ta...@apache.org on 2018/06/07 17:12:27 UTC
impala git commit: IMPALA-7143: disable affected describe
formatted/extended tests
Repository: impala
Updated Branches:
refs/heads/master 725a47b3f -> a0257013d
IMPALA-7143: disable affected describe formatted/extended tests
Change-Id: I47651e22881e9f561ce195b45e00c47d9f3c636a
Reviewed-on: http://gerrit.cloudera.org:8080/10635
Reviewed-by: Tim Armstrong <ta...@cloudera.com>
Tested-by: Tim Armstrong <ta...@cloudera.com>
Project: http://git-wip-us.apache.org/repos/asf/impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/a0257013
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/a0257013
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/a0257013
Branch: refs/heads/master
Commit: a0257013df735141e5d36f884305af26c92b7c4b
Parents: 725a47b
Author: Tim Armstrong <ta...@cloudera.com>
Authored: Thu Jun 7 08:29:38 2018 -0700
Committer: Tim Armstrong <ta...@cloudera.com>
Committed: Thu Jun 7 17:10:58 2018 +0000
----------------------------------------------------------------------
.../apache/impala/analysis/AuthorizationTest.java | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/impala/blob/a0257013/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
----------------------------------------------------------------------
diff --git a/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java b/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
index 8244e96..a1ceb2e 100644
--- a/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
+++ b/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
@@ -1960,14 +1960,16 @@ public class AuthorizationTest extends FrontendTestBase {
// Verify EXTENDED output contains all columns and data
result = fe_.describeTable(new TTableName("functional","alltypesagg"),
TDescribeOutputStyle.EXTENDED, USER);
- verifyOutputWithOptionalData(EXPECTED_DESCRIBE_EXTENDED_ALLTYPESAGG,
- resultToStringList(result));
+ // TODO: re-enable once IMPALA-7143 is fixed
+ // verifyOutputWithOptionalData(EXPECTED_DESCRIBE_EXTENDED_ALLTYPESAGG,
+ // resultToStringList(result));
// Verify FORMATTED output contains all columns and data
result = fe_.describeTable(new TTableName("functional","alltypesagg"),
TDescribeOutputStyle.FORMATTED, USER);
- verifyOutputWithOptionalData(EXPECTED_DESCRIBE_EXTENDED_ALLTYPESAGG,
- resultToStringList(result));
+ // TODO: re-enable once IMPALA-7143 is fixed
+ // verifyOutputWithOptionalData(EXPECTED_DESCRIBE_EXTENDED_ALLTYPESAGG,
+ // resultToStringList(result));
// Verify MINIMAL describe on restricted table shows limited columns.
result = fe_.describeTable(new TTableName("functional","alltypessmall"),
@@ -1978,8 +1980,9 @@ public class AuthorizationTest extends FrontendTestBase {
// Verify FORMATTED output contains all columns and data
result = fe_.describeTable(new TTableName("functional","alltypessmall"),
TDescribeOutputStyle.FORMATTED, USER);
- verifyOutputWithOptionalData(EXPECTED_DESCRIBE_EXTENDED_ALLTYPESSMALL,
- resultToStringList(result));
+ // TODO: re-enable once IMPALA-7143 is fixed
+ // verifyOutputWithOptionalData(EXPECTED_DESCRIBE_EXTENDED_ALLTYPESSMALL,
+ // resultToStringList(result));
}
// Compares two arrays but skips an expected value that contains '*' since we need to