So, one of the problems I've run into migrating the 1.x code to JDK8
is that we still have Mock in the API. That was removed in 2.x.
However, in 1.9, MockConfiguration extends a non-public API,
AccumuloConfiguration, which uses non-public Guava types for
Predicates. Code-modernization/quality checks performed by the
modernizer-maven-plugin catch the use of Guava's Predicate. I'm still
looking at this, to see if I can work around it without breaking
anything, but it's a bit frustrating, especially since the right fix
(removal of Mock) was already done in 2.x.

I'll revisit next week after a long weekend. In the meantime, if
anybody is having second thoughts about a 1.10 release, or opinions
about what to do, feel free to express them here. One option is to
simply disable the modernizer-maven-plugin and ignore those checks...
but I don't really like the idea of disabling one of our tools that
does quality checks (even if these are very minor quality items).

On Thu, Nov 7, 2019 at 4:20 PM Christopher <ctubb...@apache.org> wrote:
>
> As agreed in the recent [VOTE] thread, we will be releasing a 1.10.0
> that bumps the minimum runtime Java version to 8.
>
> I am beginning to work on getting the branch (currently still named
> 1.9) ready for release in accordance with this plan. As such, I will
> be preparing a pull request that bumps the Java version and resolves
> any errors generated by our plugins which do quality checks
> (specifically modernizer, but also some compiler warnings).
>
> However, there are still a few other outstanding issues that were
> previously labeled for 1.9, which have not yet been resolved. At least
> one of these was labeled a "blocker". These are:
> https://github.com/apache/accumulo/issues?q=is%3Aopen+project%3Aapache%2Faccumulo%2F8
>
> If anybody is able to work on these, it will be very helpful in
> getting the 1.10.0 ready for a release vote.
>
> Thanks,
>
> Christopher

Reply via email to