You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by Quanlong Huang <hu...@gmail.com> on 2018/08/30 09:37:42 UTC
Build failed in 2.x and 2.12.0-release due to updates of sentry-1.5.1-cdh5.16.0-SNAPSHOT
Hi all,
Recently, Sentry in version 1.5.1-cdh5.16.0-SNAPSHOT is updated, which
breaks the build of 2.x branch and even 2.12.0-release. The error is
[ERROR]
/tmp/jenkins/workspace/impala-compile-and-publish/fe/src/main/java/org/apache/impala/catalog/AuthorizationPolicy.java:[58,8]
org.apache.impala.catalog.AuthorizationPolicy is not abstract and does not
override abstract method
listPrivileges(java.util.Set<java.lang.String>,java.util.Set<java.lang.String>,org.apache.sentry.core.common.ActiveRoleSet)
in org.apache.sentry.provider.cache.PrivilegeCache
My question is why we depend on CDH SNAPSHOT versions even in a release? I
think the dependencies of a release should be frozen. However, there're no
stable version of cdh dependencies downloadable in
https://native-toolchain.s3.amazonaws.com. Could you help to upload stable
version of these cdh dependencies?
Thanks,
Quanlong
Re: Build failed in 2.x and 2.12.0-release due to updates of sentry-1.5.1-cdh5.16.0-SNAPSHOT
Posted by Quanlong Huang <hu...@gmail.com>.
Thank you, Fredy!
We should release a compilable 2.12.1 to replace 2.12.0 by whether
cherry-pick this patch or downgrade the CDH dependencies to cdh5.15 (which
already contains Impala-2.12 thus is compatible with 2.12). I filed a JIRA
for this (IMPALA-7518).
I'm in favor with the second choice since future changes in
cdh5.16.0-SNAPSHOT won't break the builds.
Thanks,
Quanlong
On Fri, Aug 31, 2018 at 3:49 AM Fredy Wijaya <fw...@cloudera.com> wrote:
> Hi Quanglong,
>
> I'm working on a fix: https://issues.apache.org/jira/browse/IMPALA-7513.
> CR
> is here: https://gerrit.cloudera.org/c/11362/
> I agree that the dependencies should be frozen. Feel free to file a JIRA
> for the 2.x branch to use the released versions instead of the SNAPSHOT
> versions to make the build more stable.
>
> On Thu, Aug 30, 2018 at 4:38 AM Quanlong Huang <hu...@gmail.com>
> wrote:
>
> > Hi all,
> >
> > Recently, Sentry in version 1.5.1-cdh5.16.0-SNAPSHOT is updated, which
> > breaks the build of 2.x branch and even 2.12.0-release. The error is
> >
> > [ERROR]
> >
> >
> /tmp/jenkins/workspace/impala-compile-and-publish/fe/src/main/java/org/apache/impala/catalog/AuthorizationPolicy.java:[58,8]
> > org.apache.impala.catalog.AuthorizationPolicy is not abstract and does
> not
> > override abstract method
> >
> >
> listPrivileges(java.util.Set<java.lang.String>,java.util.Set<java.lang.String>,org.apache.sentry.core.common.ActiveRoleSet)
> > in org.apache.sentry.provider.cache.PrivilegeCache
> >
> > My question is why we depend on CDH SNAPSHOT versions even in a release?
> I
> > think the dependencies of a release should be frozen. However, there're
> no
> > stable version of cdh dependencies downloadable in
> > https://native-toolchain.s3.amazonaws.com. Could you help to upload
> stable
> > version of these cdh dependencies?
> >
> > Thanks,
> > Quanlong
> >
>
Re: Build failed in 2.x and 2.12.0-release due to updates of sentry-1.5.1-cdh5.16.0-SNAPSHOT
Posted by Fredy Wijaya <fw...@cloudera.com>.
Hi Quanglong,
I'm working on a fix: https://issues.apache.org/jira/browse/IMPALA-7513. CR
is here: https://gerrit.cloudera.org/c/11362/
I agree that the dependencies should be frozen. Feel free to file a JIRA
for the 2.x branch to use the released versions instead of the SNAPSHOT
versions to make the build more stable.
On Thu, Aug 30, 2018 at 4:38 AM Quanlong Huang <hu...@gmail.com>
wrote:
> Hi all,
>
> Recently, Sentry in version 1.5.1-cdh5.16.0-SNAPSHOT is updated, which
> breaks the build of 2.x branch and even 2.12.0-release. The error is
>
> [ERROR]
>
> /tmp/jenkins/workspace/impala-compile-and-publish/fe/src/main/java/org/apache/impala/catalog/AuthorizationPolicy.java:[58,8]
> org.apache.impala.catalog.AuthorizationPolicy is not abstract and does not
> override abstract method
>
> listPrivileges(java.util.Set<java.lang.String>,java.util.Set<java.lang.String>,org.apache.sentry.core.common.ActiveRoleSet)
> in org.apache.sentry.provider.cache.PrivilegeCache
>
> My question is why we depend on CDH SNAPSHOT versions even in a release? I
> think the dependencies of a release should be frozen. However, there're no
> stable version of cdh dependencies downloadable in
> https://native-toolchain.s3.amazonaws.com. Could you help to upload stable
> version of these cdh dependencies?
>
> Thanks,
> Quanlong
>