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 <fwij...@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 <huangquanl...@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 > > >
