Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12708 )
Change subject: IMPALA-6802 (part 1): Clean up authorization tests ...................................................................... Patch Set 1: Hi Fredy, I'm going to pick your patch of COMMENT ON COLUMN to branch 2.x. However, the tests in it are in fe/src/test/java/org/apache/impala/analysis/AuthorizationStmtTest.java. This file doesn't exist in branch 2.x. It's introduce by your 6 patches of IMPALA-6802. So I'd like to cherry-pick these 6 patches first. Do you have any concerns? -- To view, visit http://gerrit.cloudera.org:8080/12708 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: comment Gerrit-Change-Id: I9cd5713607c423f644451af5ebb3494d3a728e3b Gerrit-Change-Number: 12708 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Comment-Date: Sat, 09 Mar 2019 02:53:54 +0000 Gerrit-HasComments: No
