Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15841 )
Change subject: KUDU-3090: Native owner metadata in Kudu ...................................................................... Patch Set 9: (1 comment) Should we constrain the maximum owner length to match Impala/HMS size restrictions to prevent any compatibility issues? https://github.com/apache/impala/blob/31a48610fc30db0a1f329b64deda839c70d63d1d/fe/src/main/java/org/apache/impala/analysis/AlterDbSetOwnerStmt.java#L50-L53 http://gerrit.cloudera.org:8080/#/c/15841/6/src/kudu/integration-tests/hms_itest-base.h File src/kudu/integration-tests/hms_itest-base.h: http://gerrit.cloudera.org:8080/#/c/15841/6/src/kudu/integration-tests/hms_itest-base.h@79 PS6, Line 79: own nit: owner -- To view, visit http://gerrit.cloudera.org:8080/15841 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I67f5bfdf56d409960365fd5803913a2d3800831d Gerrit-Change-Number: 15841 Gerrit-PatchSet: 9 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Mon, 06 Jul 2020 12:55:13 +0000 Gerrit-HasComments: Yes
