Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
HBase >> mail # dev >> HBASE Code format


Copy link to this message
-
Re: HBASE Code format
+1 for 100 chars too.

Though really, as long as we have a standard that we stick to, doesn't
really matter to me.

-------------------
Jesse Yates
On Thu, Mar 22, 2012 at 11:06 AM, Todd Lipcon <[EMAIL PROTECTED]> wrote:

> +1 for 100 chars. I think unlimited gets messy.
>
> On Thu, Mar 22, 2012 at 9:50 AM, Dave Wang <[EMAIL PROTECTED]> wrote:
> > +1 on Lars's comment.
> >
> > On Thu, Mar 22, 2012 at 9:00 AM, lars hofhansl <[EMAIL PROTECTED]>
> wrote:
> >
> >> Put me in the I-couldn't-care-less camp :) 80, 100, 120, or even no
> limit
> >> is fine with me.
> >> Would personally prefer no limit. Instead leave it up to the good taste
> of
> >> the contributors and us committers to format the code in the most
> readable
> >> way.
> >>
> >>
> >> -- Lars
> >>
> >>
> >> ________________________________
> >>  From: Laxman <[EMAIL PROTECTED]>
> >> To: [EMAIL PROTECTED]
> >> Sent: Wednesday, March 21, 2012 10:17 PM
> >> Subject: HBASE Code format
> >>
> >> Hi Devs,
> >>
> >> How about raising the "max line width" from 80 (to 100 or 120)?
> >> IMO, 80 characters length is too low & it makes the code bit ugly.
> >>
> >> Example:
> >>     long timstamp = conf.getLong(TIMESTAMP_CONF_KEY,
> >> System.currentTimeMillis());
> >>
> >> Above code snippet after formatting, it turned to
> >>
> >>     long timstamp = conf
> >>         .getLong(TIMESTAMP_CONF_KEY, System.currentTimeMillis());
> >>
> >> Please respond with your opinion considering the following points.
> >>
> >> - Sun Java coding standards drafted in 1999
> >> - Terminals(Monitors) we are using now are very wider and 80 characters
> is
> >> not a valid limit anymore.
> >> - As per Ted, Google raised this limit
> >> [https://issues.apache.org/jira/browse/HBASE-5564]
> >>
> >> Note: We don't need to reformat entire codebase. My proposal is to apply
> >> these standards to new code getting commited.
> >>
> >> --
> >> Regards,
> >> Laxman
> >>
>
>
>
> --
> Todd Lipcon
> Software Engineer, Cloudera
>