Adam Holley has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/11469 )
Change subject: IMPALA-7591: Use short name for setting owner ...................................................................... IMPALA-7591: Use short name for setting owner This patch changes the owner to be the shortname in a Kerberized cluster. Authorization with Sentry is done using the shortname and Hive uses the shortname. Because object ownership privileges are based on owner, authorization will not work with the full name. The IOException for org.apache.impala.authorization.User.getShortName should only ever be thrown if the format for the name is wrong. i.e. It's actually a BadFormatString exception. Testing: - Ran all frontend, and e2e tests. - Manually verified on kerberized cluster. Change-Id: Iab392a0cd5c0bb98442e1f7ac39f470310a21f11 --- M fe/src/main/java/org/apache/impala/analysis/AlterViewStmt.java M fe/src/main/java/org/apache/impala/analysis/Analyzer.java M fe/src/main/java/org/apache/impala/analysis/CreateDbStmt.java M fe/src/main/java/org/apache/impala/analysis/CreateTableLikeStmt.java M fe/src/main/java/org/apache/impala/analysis/CreateTableStmt.java M fe/src/main/java/org/apache/impala/analysis/CreateViewStmt.java 6 files changed, 14 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/69/11469/4 -- To view, visit http://gerrit.cloudera.org:8080/11469 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iab392a0cd5c0bb98442e1f7ac39f470310a21f11 Gerrit-Change-Number: 11469 Gerrit-PatchSet: 4 Gerrit-Owner: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>