Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@accumulo.apache.org
Received-SPF: pass (athena.apache.org: domain of josh.elser@gmail.com
 designates 209.85.192.45 as permitted sender)
Message-ID: <54AD95B2.5020804@gmail.com>
Date: Wed, 07 Jan 2015 15:23:14 -0500
From: Josh Elser <josh.elser@gmail.com>
User-Agent: Postbox 3.0.11 (Macintosh/20140602)
MIME-Version: 1.0
To: dev@accumulo.apache.org
Subject: Re: [VOTE][LAZY] Format all supported branches
References: 
 <CAL5zq9YdKBWeYiyRqFTR2isHB7wjK-5cQ-FHyimKAa78EFDG8A@mail.gmail.com>
 <CADczPYQiphWecmYJjgAF=9ZCc0_v+RZZD9uh2GctOagbjiJYrQ@mail.gmail.com>
In-Reply-To: 
 <CADczPYQiphWecmYJjgAF=9ZCc0_v+RZZD9uh2GctOagbjiJYrQ@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

ack'ed

John Vines wrote:
> +1
>
> On Wed, Jan 7, 2015 at 3:12 PM, Christopher<ctubbsii@apache.org>  wrote:
>
>> To make it easier to apply some minimal checkstyle rules for ACCUMULO-3451,
>> I'm announcing my intentions to do a full, one-time, auto-format and
>> organize imports on all our supported branches (1.5, 1.6, and master) to
>> bring us up to some degree of compliance with our agreed-upon formatting
>> standards.
>>
>> Benefits:
>> To have additional checks, in particular against javadoc problems and other
>> common trivial warnings in the build.
>> To ensure less divergence from our agreed-upon formatting standards.
>> Formatting first makes it much less tedious and easier on me to add these
>> checks to the build.
>>
>> Issues I've considered:
>> I will deal with all the merge conflicts.
>> I will ignore generated thrift code.
>> Conflicts with new code in people's branches should be minimal (and easily
>> resolved by formatting according to our standards).
>> Regarding concerns about history tracking, in general, each format change
>> is small, but they are numerous. So, the impact on tracking history should
>> be very minimal (you'll see things like a brace moved to the same line as
>> the else statement it is associated with... stuff that won't generally
>> affect your ability to debug).
>> I'll also do a "format only" commit, separately from any substantive
>> changes regarding the rule changes, so the mass formatting change will
>> happen in one place, and it will also be easy to revert, if absolutely
>> necessary.
>>
>> I'll give this 24 hours (it can be reverted if somebody objects after
>> that).
>>
>> --
>> Christopher L Tubbs II
>> http://gravatar.com/ctubbsii
>>
>