Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-07-21 Thread Akira Ajisaka
Based on the positive feedback, filed https://issues.apache.org/jira/browse/HADOOP-17813 to update the checkstyle rule. I don't think it requires a vote thread. Thanks and regards, Akira On Sun, Jun 13, 2021 at 1:14 AM Steve Loughran wrote: > > +1 > > if you look closely the hadoop-azure module

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-06-12 Thread Steve Loughran
+1 if you look closely the hadoop-azure module went to 100 lines a while back and all is good On Wed, 19 May 2021 at 22:13, Sean Busbey wrote: > Hello! > > What do folks think about changing our line length guidelines to allow for > 100 character width? > > Currently, we tell folks to follow th

Re: [E] Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-06-12 Thread Szilard Nemeth
Thanks for this initiative, Sean. +1 for increasing the length to 100 characters. I can't see a VOTE thread regarding this subject. Am I missing something? Best, Szilard On Mon, May 24, 2021 at 11:49 PM Jonathan Eagles wrote: > In apache tez, formal line length is 120 characters. So, I recomm

Re: [E] Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-24 Thread Jonathan Eagles
In apache tez, formal line length is 120 characters. So, I recommend 120+ On Mon, May 24, 2021 at 4:46 PM Kihwal Lee wrote: > +1 for the 100 char limit. > But I would have liked 132 columns more. :) > > Kihwal > > On Mon, May 24, 2021 at 1:46 PM Sean Busbey > wrote: > > > Hi folks! > > > > The

Re: [E] Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-24 Thread Kihwal Lee
+1 for the 100 char limit. But I would have liked 132 columns more. :) Kihwal On Mon, May 24, 2021 at 1:46 PM Sean Busbey wrote: > Hi folks! > > The consensus seems pretty strongly in favor of increasing the line length > limit. Do folks still want to see a formal VOTE thread? > > > > On May 1

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-24 Thread Sean Busbey
Hi folks! The consensus seems pretty strongly in favor of increasing the line length limit. Do folks still want to see a formal VOTE thread? > On May 19, 2021, at 4:22 PM, Sean Busbey wrote: > > Hello! > > What do folks think about changing our line length guidelines to allow for > 100 cha

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread John Zhuge
; > > > > How > > > > > > > we are planning to handle this? > > > > > > > > > > > > > > On Thu, May 20, 2021, 11:09 AM Qi Zhu <821684...@qq.com > > > > > > 821684...@qq.com>> wrote: > > >

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Takanobu Asanuma
> > > > > > > On Thu, May 20, 2021, 11:09 AM Qi Zhu <821684...@qq.com > > > > 821684...@qq.com>> wrote: > > > > > > > > > > > >> +1 100 is reasonable. > > > > > >> > > > > >

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Chao Sun
wrote: > > > > > > > > > >> +1 100 is reasonable. > > > > >> > > > > >> > > > > >> > > > > >> ---Original--- > > > > >> From: "Xiaoqiao He" > > hexiaoq...@apache.org >

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Sangjin Lee
; > > > >> > > > >> ---Original--- > > > >> From: "Xiaoqiao He" > hexiaoq...@apache.org > > > >> > > > >> Date: Thu, May 20, 2021 13:35 PM > > > >> To: "Masatake Iwasaki" > > iwasak...@oss.nttda

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Gergely Pollak
;> From: "Xiaoqiao He" hexiaoq...@apache.org > > >> > > >> Date: Thu, May 20, 2021 13:35 PM > > >> To: "Masatake Iwasaki" > iwasak...@oss.nttdata.co.jp>>; > > >> Cc: "Akira Ajisaka"mailto:aajis...@apache.org > &

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Vivek Ratnavel
ailto:common-dev@hadoop.apache.org > >>;"Hdfs-dev" hdfs-...@hadoop.apache.org> > >> >;"yarn-dev" yarn-...@hadoop.apache.org>>;"mapreduce-dev"< > >> mapreduce-...@hadoop.apache.org <mailto:mapreduce-...@hadoop.apache.org > >&

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Sean Busbey
-dev"> <mailto:hdfs-...@hadoop.apache.org> >> >;"yarn-dev"> <mailto:yarn-...@hadoop.apache.org>>;"mapreduce-dev"< >> mapreduce-...@hadoop.apache.org <mailto:mapreduce-...@hadoop.apache.org>>; >> Subject: Re: [DISCUSS] Change proje

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Viraj Jasani
t; From: "Xiaoqiao He" > > Date: Thu, May 20, 2021 13:35 PM > > > To: "Masatake Iwasaki" > > Cc: "Akira Ajisaka" > > common-dev@hadoop.apache.org>;"Hdfs-dev" > > >;"yarn-dev" > > mapreduce-...@hadoop.a

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-20 Thread Bhavik Patel
1 13:35 PM > To: "Masatake Iwasaki" Cc: "Akira Ajisaka" common-dev@hadoop.apache.org>;"Hdfs-dev" >;"yarn-dev" mapreduce-...@hadoop.apache.org>; > Subject: Re: [DISCUSS] Change project style guidelines to allow line > length 100 > >

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-19 Thread Qi Zhu
+1 100 is reasonable. ---Original--- From: "Xiaoqiao He"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

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-19 Thread Xiaoqiao He
+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/7813c2f8a49b

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-19 Thread Masatake Iwasaki
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 I have no inconvenience on 100 characters for

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-19 Thread Duo Zhang
+1 for 100 instead of 80. I can see that some projects have already changed to 120. Akira Ajisaka 于2021年5月20日周四 上午10:01写道: > I'm +1 to allow <= 100 chars. > > FYI: There were some discussions long before: > - > https://lists.apache.org/thread.html/7813c2f8a49b1d1e7655dad180f2d915a280b2f4d562cfe

Re: [DISCUSS] Change project style guidelines to allow line length 100

2021-05-19 Thread Akira Ajisaka
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/3e1785cbbe14dcab9bb970fa0f534811cfe