Dan Burkert has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/11601 )
Change subject: Bump Sentry and Hadoop versions
......................................................................
Bump Sentry and Hadoop versions
The motivation is to get access to SENTRY-2371, which introduces a new
Thrift interface that will be used by Kudu to retrieve user privileges.
SENTRY-2371 has not been released yet, so the new Sentry version is
based on the latest commit of the master branch. The new Sentry version
appears to have an incompatibility with Hadoop 2.8.2, so this also
includes a bump to the latest Hadoop 2.8.x version. I suspect the fix
between 2.8.2 and 2.8.5 was introduced in [1], but the commit message is
vague and I haven't dug any further. The incompatiblity manifests as an
exception during Sentry startup:
java.lang.NoSuchMethodError:
org.apache.hadoop.conf.Configuration.addResource(Ljava/net/URL;Z)V
at
org.apache.sentry.service.thrift.SentryService.loadConfig(SentryService.java:576)
at
org.apache.sentry.service.thrift.SentryService$CommandImpl.run(SentryService.java:600)
at org.apache.sentry.SentryMain.main(SentryMain.java:120)
[1]:
https://github.com/apache/hadoop/commit/7af9b8ad1e993ef791aa38740b6aabc4c233a30f
Change-Id: I8bcc4ff6fac0435b037b984f45da75bed6ff4be5
Reviewed-on: http://gerrit.cloudera.org:8080/11601
Reviewed-by: Hao Hao <[email protected]>
Tested-by: Kudu Jenkins
---
M src/kudu/sentry/sentry_common_service.thrift
M src/kudu/sentry/sentry_policy_service.thrift
M thirdparty/vars.sh
3 files changed, 99 insertions(+), 10 deletions(-)
Approvals:
Hao Hao: Looks good to me, approved
Kudu Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/11601
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8bcc4ff6fac0435b037b984f45da75bed6ff4be5
Gerrit-Change-Number: 11601
Gerrit-PatchSet: 3
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
