Yes. I will check with Sergio once he is available to see if we can combine
those two definitions into one. Otherwise, People may be confused and it is
easier to make mistake which one to use. I suspect UNUSED_PATH_UPDATE_IMG_NUM
is used for permission update to mark it as not valid.
On Thu, Aug 10, 2017 at 7:42 PM, Alexander Kolbasov <ak...@cloudera.com>
wrote:
> One of this is part of thrift definitions:
>
> # Value used to specify that a path image number is not used on a
> request or response
> const i64 UNUSED_PATH_UPDATE_IMG_NUM = -1;
>
> The default value is -1
>
> Since you raised the question - can you investigate this deeper?
>
>
> Thanks,
>
>
> - Sasha
>
>
> On Thu, Aug 10, 2017 at 2:18 PM, Na Li <lina...@cloudera.com> wrote:
>
> > Hi,
> >
> > I found multiple definitions of unused image number.
> >
> > Can we just remove
> > "sentry_hdfs_serviceConstants.UNUSED_PATH_UPDATE_IMG_NUM" and keep
> > "ServiceConstants.IMAGE_NUMBER_UPDATE_UNINITIALIZED"?
> >
> > public class sentry_hdfs_serviceConstants {
> >
> > public static final long UNUSED_PATH_UPDATE_IMG_NUM = -1L;
> > }
> >
> > ServiceConstants.IMAGE_NUMBER_UPDATE_UNINITIALIZED = 0L;
> >
> > Thanks,
> >
> > Lina
> >
>
