You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Hadoop QA (JIRA)" <ji...@apache.org> on 2017/07/03 04:59:00 UTC

[jira] [Commented] (HBASE-12527) Remove padding around parens

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

Hadoop QA commented on HBASE-12527:
-----------------------------------

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m  0s{color} | {color:blue} Docker mode activated. {color} |
| {color:red}-1{color} | {color:red} patch {color} | {color:red}  0m  4s{color} | {color:red} HBASE-12527 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/0.4.0/precommit-patchnames for help. {color} |
\\
\\
|| Subsystem || Report/Notes ||
| JIRA Issue | HBASE-12527 |
| JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12684273/HBASE-12527-v2.patch |
| Console output | https://builds.apache.org/job/PreCommit-HBASE-Build/7470/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> Remove padding around parens
> ----------------------------
>
>                 Key: HBASE-12527
>                 URL: https://issues.apache.org/jira/browse/HBASE-12527
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Sean Busbey
>            Assignee: Varun Saxena
>            Priority: Minor
>         Attachments: HBASE-12527-0.98.patch, HBASE-12527-branch-1.patch, HBASE-12527.patch, HBASE-12527-v2.patch
>
>
> We have checkstyle [enforcing no padding around parenthesis|http://checkstyle.sourceforge.net/config_whitespace.html#ParenPad] and that matches [our ref guide|http://hbase.apache.org/book.html#common.patch.feedback.space.invaders].
> Right now we have ~150 instances of not meeting this standard.
> list impacted files:
> {code}
> grep -rl -E "(\( |[^ ] \))" */src/* | grep java | grep -v generated
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)