I would like to retain the current limit of 80 characters. Increasing or decreasing will not really help as whatever problem of line wrap you are facing today, you will face that even when the limit is 60 or 100 or 120. So no matter what limit we set, there will always be reasons to not like that limit.
That said, I would not like to remove the line length rule completely as it help to make the code readable. Thanks, Dinesh On Fri, Dec 13, 2019 at 3:37 AM Attila Doroszlai <[email protected]> wrote: > Hi All, > > I would like to start a discussion on whether we should change the > line length limit of 80 characters (enforced via Checkstyle) in Ozone. > I think it was a necessity in the Hadoop repo, but now that Ozone is > in its own repo, we no longer need to stick to 80 chars. > > I would like to ask for your opinion if we should: > > 1. keep the current limit of 80 characters > 2. impose a higher limit (please suggest value) > 3. remove the line length rule completely > > Thanks. > > -Attila > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
