hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matteo Bertozzi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-12525) Wrap lines longer than 100 chars
Date Thu, 20 Nov 2014 14:39:33 GMT

    [ https://issues.apache.org/jira/browse/HBASE-12525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14219432#comment-14219432
] 

Matteo Bertozzi commented on HBASE-12525:
-----------------------------------------

{quote}My concern is that there are so many warnings from checkstyle{quote}
we can do the same thing we have for javadoc now, where we set the current number of errors
and we show only the diff in the result. You want enforce that the new stuff is correct. (it
is almost the same thing as the deprecated api.. you enforce the new way, but you keep the
old bad stuff to avoid breaking everyone who can't migrate)

{quote}Would your concern be mitigated if I also brought the fix back to branch-1? earlier
branches?{quote}
assuming that everyone can rebase up to this patch... yes it will help. but if you are stuck
in an old branch and you want to backport stuff this is just extra work and it will be increasingly
painful as more as you differ from the original branch.

> Wrap lines longer than 100 chars
> --------------------------------
>
>                 Key: HBASE-12525
>                 URL: https://issues.apache.org/jira/browse/HBASE-12525
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Sean Busbey
>            Assignee: Sean Busbey
>            Priority: Minor
>              Labels: checkstyle
>
> After broadening the acceptable long lines in HBASE-12523, we still end up with ~900
cases of lines over our chosen length.
> Go through and wrap all of those that are sensible to wrap and then update HBASE-12523
with a list of additional exclusions.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message