Return-Path: X-Original-To: apmail-accumulo-dev-archive@www.apache.org Delivered-To: apmail-accumulo-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 80E52173BD for ; Mon, 6 Oct 2014 20:57:25 +0000 (UTC) Received: (qmail 7441 invoked by uid 500); 6 Oct 2014 20:57:25 -0000 Delivered-To: apmail-accumulo-dev-archive@accumulo.apache.org Received: (qmail 7399 invoked by uid 500); 6 Oct 2014 20:57:25 -0000 Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list dev@accumulo.apache.org Received: (qmail 7387 invoked by uid 99); 6 Oct 2014 20:57:24 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 06 Oct 2014 20:57:24 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of mdrob@cloudera.com designates 209.85.218.44 as permitted sender) Received: from [209.85.218.44] (HELO mail-oi0-f44.google.com) (209.85.218.44) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 06 Oct 2014 20:57:21 +0000 Received: by mail-oi0-f44.google.com with SMTP id x69so4229259oia.3 for ; Mon, 06 Oct 2014 13:57:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:content-type; bh=yKc7zPelcILHDfD5PcStkO1NhFduqoj91uJREj6Rom4=; b=KaBHgWKV/vwOxCbw8uIyhIZRUURfyQ4yGPPVYy3Aq+nf2YGe6O6+30PQg47O3odPYY ZS4PGvdElBy6yIqy86KvhZpDRBpWVf2EOR9I/3tZua9TXv4WibKwFerBTckGithPxSpR l0HmrXjtILYhaLuPqVNFZlS/F6lEKu80WnUEUV6BK/OratPOqfr5hpaU4H6JQDLM+Kke 5a0sT73nq4AFgRt2Bss0oeoQGPHESz+g+Md9yKv3SdEWiru8qOVAApeWUBganb9y9i9h bxZUXM+V3uMm2ikgQfPGF0jSnkzCH7QBwvOF1lVefnoYjFaS5WRLN962oxflVOzJ2QdK 4Zbg== X-Gm-Message-State: ALoCoQmddBQy66OOAdVcHQJlmdnXjUUTNe6joEHckVf3T2RwSI+ZlvXIul7oXxe09ItVifzNGhrp X-Received: by 10.60.42.178 with SMTP id p18mr31154517oel.15.1412629020409; Mon, 06 Oct 2014 13:57:00 -0700 (PDT) MIME-Version: 1.0 Received: by 10.60.222.105 with HTTP; Mon, 6 Oct 2014 13:56:40 -0700 (PDT) In-Reply-To: References: From: Mike Drob Date: Mon, 6 Oct 2014 15:56:40 -0500 Message-ID: Subject: Re: Deprecation removal for 1.7.0 To: Accumulo Dev List Content-Type: multipart/alternative; boundary=001a11c201c6f6185f0504c7522e X-Virus-Checked: Checked by ClamAV on apache.org --001a11c201c6f6185f0504c7522e Content-Type: text/plain; charset=UTF-8 Do we still have mapred(uce) stuff? On Mon, Oct 6, 2014 at 3:54 PM, Christopher 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 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 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 > > > --001a11c201c6f6185f0504c7522e--