+1 for 100 characters. Thanks. 2021年5月21日(金) 8:38 Chao Sun <sunc...@apache.org>:
> +1. I think this will be very helpful. > > On Thu, May 20, 2021 at 3:10 PM Sangjin Lee <sj...@apache.org> wrote: > > > +1 (binding). It's long overdue IMHO. > > > > On Thu, May 20, 2021 at 2:11 PM Gergely Pollak > > <shuzi...@cloudera.com.invalid> wrote: > > > > > I really like this initiative, thank you! > > > +1 for line length increase to 100 characters. > > > > > > Regards, > > > Gergely Pollak > > > > > > On Thu, May 20, 2021 at 8:30 PM Vivek Ratnavel < > vivekratna...@apache.org > > > > > > wrote: > > > > > > > +1 (non-binding) to increase line length to 100 characters. This will > > > > definitely help clear most of the checkstyle violations. > > > > > > > > Thank you Sean for starting this thread! > > > > > > > > On Thu, May 20, 2021 at 7:17 AM Sean Busbey > <sbus...@apple.com.invalid > > > > > > > wrote: > > > > > > > > > Hi Bhavik! > > > > > > > > > > What concerns do you have about back porting patches to earlier > > release > > > > > branches? > > > > > > > > > > If we change our style guidelines then presumably we can do that > for > > > all > > > > > branches, so a backport from e.g. trunk to branch-3.3 won’t fail a > > > style > > > > > check on the destination branch unless something changed in the > > > > backporting. > > > > > > > > > > If you are referring to patches for clearing up line length > > violations, > > > > my > > > > > usual preference is to aim for my changes to be on all active > release > > > > > lines. So at least in the case of the patches coming from me or > being > > > > > committed by me, there’d be effort to make sure all branches end up > > as > > > > easy > > > > > to backport to as they were prior to the clean up. > > > > > > > > > > > > > > > > > > > > > On May 20, 2021, at 2:27 AM, Bhavik Patel < > > bhavikpatel...@gmail.com> > > > > > wrote: > > > > > > > > > > > > I am just worried about the backporting of the Jira to child > > branch!! > > > > How > > > > > > we are planning to handle this? > > > > > > > > > > > > On Thu, May 20, 2021, 11:09 AM Qi Zhu <821684...@qq.com <mailto: > > > > > 821684...@qq.com>> wrote: > > > > > > > > > > > >> +1 100 is reasonable. > > > > > >> > > > > > >> > > > > > >> > > > > > >> ---Original--- > > > > > >> From: "Xiaoqiao He"<hexiaoq...@apache.org <mailto: > > > > hexiaoq...@apache.org > > > > > >> > > > > > >> Date: Thu, May 20, 2021 13:35 PM > > > > > >> To: "Masatake Iwasaki"<iwasak...@oss.nttdata.co.jp <mailto: > > > > > iwasak...@oss.nttdata.co.jp>>; > > > > > >> Cc: "Akira Ajisaka"<aajis...@apache.org <mailto: > > aajis...@apache.org > > > > >>;"Hadoop > > > > > Common"< > > > > > >> common-...@hadoop.apache.org <mailto: > common-...@hadoop.apache.org > > > > > >>;"Hdfs-dev"<hdfs-dev@hadoop.apache.org <mailto: > > > > > hdfs-dev@hadoop.apache.org> > > > > > >> >;"yarn-dev"<yarn-...@hadoop.apache.org <mailto: > > > > > yarn-...@hadoop.apache.org>>;"mapreduce-dev"< > > > > > >> mapreduce-...@hadoop.apache.org <mailto: > > > > mapreduce-...@hadoop.apache.org > > > > > >>; > > > > > >> Subject: Re: [DISCUSS] Change project style guidelines to allow > > line > > > > > >> length 100 > > > > > >> > > > > > >> > > > > > >> +1 for <= 100 chars long per line length. > > > > > >> > > > > > >> On Thu, May 20, 2021 at 10:28 AM Masatake Iwasaki < > > > > > >> iwasak...@oss.nttdata.co.jp> wrote: > > > > > >> > > > > > >> > I'm +1 too. > > > > > >> > I feel 80 characters limit tends to degrade readability by > > > > > introducing > > > > > >> > useless line breaks. > > > > > >> > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E > > > > > < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E > > > > > > > > > > > >> > > > > > > >> < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E> > > > > > < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E> > > > > > >> > > > > > >> ; > > > > > >> > I have no inconvenience on 100 characters for using Emacs > and > > > > > >> side-by-side > > > > > >> > diff even on 13-inch MBP. > > > > > >> > > > > > > >> > Masatake Iwasaki > > > > > >> > > > > > > >> > On 2021/05/20 11:00, Akira Ajisaka wrote: > > > > > >> > > I'm +1 to allow <= 100 chars. > > > > > >> > > > > > > > >> > > FYI: There were some discussions long before: > > > > > >> > > - > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E > > > > > < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E > > > > > > > > > > > >> > > > > > > >> < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E> > > > > > < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe981e1dd4e%401406489966%40%3Ccommon-dev.hadoop.apache.org%3E> > > > > > >>; > > > > > >> > - > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/3e1785cbbe14dcab9bb970fa0f534811cfe00795a8cd1100580f27dc%401430849118%40%3Ccommon-dev.hadoop.apache.org%3E > > > > > < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/3e1785cbbe14dcab9bb970fa0f534811cfe00795a8cd1100580f27dc%401430849118%40%3Ccommon-dev.hadoop.apache.org%3E > > > > > > > > > > > >> > > > > > > >> < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/3e1785cbbe14dcab9bb970fa0f534811cfe00795a8cd1100580f27dc%401430849118%40%3Ccommon-dev.hadoop.apache.org%3E> > > > > > < > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/3e1785cbbe14dcab9bb970fa0f534811cfe00795a8cd1100580f27dc%401430849118%40%3Ccommon-dev.hadoop.apache.org%3E> > > > > > >>; > > > > > >> > > > > > > >> > > Thanks, > > > > > >> > > Akira > > > > > >> > > > > > > > >> > > On Thu, May 20, 2021 at 6:36 AM Sean Busbey > > > > > >> <sbus...@apple.com.invalid <mailto:sbus...@apple.com.invalid > >> > > > > > >> > wrote: > > > > > >> > >> > > > > > >> > >> Hello! > > > > > >> > >> > > > > > >> > >> What do folks think about changing our line length > > > > > >> guidelines to allow > > > > > >> > for 100 character width? > > > > > >> > >> > > > > > >> > >> Currently, we tell folks to follow the sun style > > guide > > > > > with > > > > > >> some > > > > > >> > exception unrelated to line length. That guide says width > of > > 80 > > > > is > > > > > the > > > > > >> > standard and our current check style rules act as > > enforcement. > > > > > >> > >> > > > > > >> > >> Looking at the current trunk codebase our nightly > > > build > > > > > >> shows a total > > > > > >> > of ~15k line length violations; it’s about 18% of > identified > > > > > >> checkstyle > > > > > >> > issues. > > > > > >> > >> > > > > > >> > >> The vast majority of those line length violations > > are > > > <= > > > > > 100 > > > > > >> characters > > > > > >> > long. 100 characters happens to be the length for the > Google > > > Java > > > > > >> Style > > > > > >> > Guide, another commonly adopted style guide for java > > projects, > > > > so I > > > > > >> suspect > > > > > >> > these longer lines leaking past the checkstyle precommit > > > warning > > > > > >> might be a > > > > > >> > reflection of committers working across multiple java > > > codebases. > > > > > >> > >> > > > > > >> > >> I don’t feel strongly about lines being longer, > but > > I > > > > > would > > > > > >> like to > > > > > >> > move towards more consistent style enforcement as a > project. > > > > > Updating > > > > > >> our > > > > > >> > project guidance to allow for 100 character lines would > > reduce > > > > the > > > > > >> > likelihood that folks bringing in new contributions need a > > > > > precommit > > > > > >> test > > > > > >> > cycle to get the formatting correct. > > > > > >> > >> > > > > > >> > >> Does anyone feel strongly about keeping the line > > > length > > > > > >> limit at 80 > > > > > >> > characters? > > > > > >> > >> > > > > > >> > >> Does anyone feel strongly about contributions > coming > > > in > > > > > that > > > > > >> clear up > > > > > >> > line length violations? > > > > > >> > >> > > > > > >> > >> > > > > > >> > >> > > > > > >> > > > --------------------------------------------------------------------- > > > > > >> > >> To unsubscribe, e-mail: > > > > > >> common-dev-unsubscr...@hadoop.apache.org <mailto: > > > > > common-dev-unsubscr...@hadoop.apache.org> > > > > > >> > >> For additional commands, e-mail: > > > > > >> common-dev-h...@hadoop.apache.org <mailto: > > > > > common-dev-h...@hadoop.apache.org> > > > > > >> > >> > > > > > >> > > > > > > > >> > > > > > > > >> > > > --------------------------------------------------------------------- > > > > > >> > > To unsubscribe, e-mail: > > > > > common-dev-unsubscr...@hadoop.apache.org <mailto: > > > > > common-dev-unsubscr...@hadoop.apache.org> > > > > > >> > > For additional commands, e-mail: > > > > > >> common-dev-h...@hadoop.apache.org <mailto: > > > > > common-dev-h...@hadoop.apache.org> > > > > > >> > > > > > > > >> > > > > > > >> > > > > > > > --------------------------------------------------------------------- > > > > > >> > To unsubscribe, e-mail: > > > common-dev-unsubscr...@hadoop.apache.org > > > > > <mailto:common-dev-unsubscr...@hadoop.apache.org> > > > > > >> > For additional commands, e-mail: > > > > common-dev-h...@hadoop.apache.org > > > > > <mailto:common-dev-h...@hadoop.apache.org> > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > >
