Do we still have mapred(uce) stuff? On Mon, Oct 6, 2014 at 3:54 PM, Christopher <ctubb...@apache.org> wrote:
> The main thing I'm looking at which is causing problems for me is the > instance.getConfiguration() stuff. It was never well defined, usually > didn't work or do what was expected of it, and is still being leveraged > (incorrectly) by new code (replication, for instance, and I've already > informed Josh), because of > ServerConfigurationUtil.getConfiguration(Instance instance). It wasn't > formally deprecated until 1.6.0, though. > > Aside from that, everything else is just a nice cleanup. A somewhat > exhaustive list of what I was looking at was: > > Scanner timeout options > extra batchwriter/batchdeleter factory methods > some junk in MutationsRejectedException > extra ZooKeeperInstance constructors > securityOperations stuff from 1.5 > extra getSplits and flush in tableOperations > Constants.NO_AUTHS > KeyExtents.getKeyExtentsForRange > an extra Value constructor which copies from a ByteBuffer > iterators that moved packages in 1.4 > some protected getters in the mapred stuff > unused RangeInputSplit in InputFormatBase > LogFileKey/LogFileValue (old version) > > > You can review the expected changes at > https://github.com/ctubbsii/accumulo/tree/ACCUMULO-3197 (in two commits, > one for instance stuff, the other for aggregators and everything else). > > > -- > Christopher L Tubbs II > http://gravatar.com/ctubbsii > > On Mon, Oct 6, 2014 at 4:11 PM, Sean Busbey <bus...@cloudera.com> wrote: > > > No objection to removing aggregators. > > > > If anything first deprecated in 1.5 has managed to live this long in 1.7 > > I'd like to keep it so folks have an easier time getting off of 1.5 when > we > > EOL it. But I realize some things have probably already been removed. > > > > On Mon, Oct 6, 2014 at 3:00 PM, Christopher <ctubb...@apache.org> wrote: > > > > > Re: ACCUMULO-3197 > > > > > > First: > > > Any objections to finally removing Aggregators in 1.7.0? > > > They've been deprecated in favor of Combiners since 1.4. > > > > > > Second: > > > Is there any API deprecated in 1.6.x or earlier that you really want > > > preserved in 1.7.0? > > > (I know we need to keep INSTANCE_DFS_{URI,DIR} properties for volume > > > upgrades, at least.) > > > > > > -- > > > Christopher L Tubbs II > > > http://gravatar.com/ctubbsii > > > > > > > > > > > -- > > Sean > > >