Fredy Wijaya has uploaded a new patch set (#5). ( 
http://gerrit.cloudera.org:8080/10643 )

Change subject: IMPALA-7144: Re-enable TestDescribeTableResults
......................................................................

IMPALA-7144: Re-enable TestDescribeTableResults

This patch makes the TestDescribeTableResults more robust by only
comparing the information that the authorization cares about instead
of comparing all output in DESCRIBE. This change will avoid any unnecessary
changes to AuthorizationTest if HMS updates the DESCRIBE output.

The test is also updated to support standalone execution without relying
on other tests be executed first since it can cause the test to be flaky
especially if the tests in AuthorizationTest are executed in parallel.

Testing:
- Ran all FE tests

Cherry-picks: not for 2.x

Change-Id: I3aeaecf5b6d906a66d338e165a6d506e3964563f
---
M fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
1 file changed, 173 insertions(+), 186 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/43/10643/5
--
To view, visit http://gerrit.cloudera.org:8080/10643
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3aeaecf5b6d906a66d338e165a6d506e3964563f
Gerrit-Change-Number: 10643
Gerrit-PatchSet: 5
Gerrit-Owner: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Vuk Ercegovac <[email protected]>

Reply via email to