Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/11721 )
Change subject: IMPALA-7721: Fix broken /catalog_object web API when getting a privilege ...................................................................... Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/11721/4/be/src/catalog/catalog-util.cc File be/src/catalog/catalog-util.cc: http://gerrit.cloudera.org:8080/#/c/11721/4/be/src/catalog/catalog-util.cc@a214 PS4, Line 214: what is the severity of the issue? ... priv doesn't get shown, crash, something else? http://gerrit.cloudera.org:8080/#/c/11721/4/be/src/catalog/catalog-util.cc@271 PS4, Line 271: Status TPrivilegeFromObjectName(const string& object_name, TPrivilege* privilege) { is there a comparable parsing method on the java side (frontend)? if so, would be worthwhile for this to avoid duplication. http://gerrit.cloudera.org:8080/#/c/11721/4/be/src/catalog/catalog-util.cc@274 PS4, Line 274: for (auto& s: split) { const -- To view, visit http://gerrit.cloudera.org:8080/11721 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I525149d113a1437c1e1493ad3c25a755e370b0c7 Gerrit-Change-Number: 11721 Gerrit-PatchSet: 4 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com> Gerrit-Comment-Date: Thu, 18 Oct 2018 18:40:13 +0000 Gerrit-HasComments: Yes