You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by "László Bodor (Jira)" <ji...@apache.org> on 2020/04/13 10:15:00 UTC

[jira] [Commented] (TEZ-4144) Checkstyle: '{' is followed by whitespace

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

László Bodor commented on TEZ-4144:
-----------------------------------

I'm using an eclipse formatter, which is accepted in hive project. It auto-formats my sources in a way that tez precommit doesn't like, more specifically it would not accept a white space after the opening curly bracket:
{code}
{ "vertex (+task stats: all/succeeded/failed/killed)", "node", "status", "numAttempts" }
{code}
is it intentional? that warning doesn't make sense to me, moreover, I prefer a space after "{". 
cc: [~jeagles], could I change checkstyle config in order to ignore this?

> Checkstyle: '{' is followed by whitespace
> -----------------------------------------
>
>                 Key: TEZ-4144
>                 URL: https://issues.apache.org/jira/browse/TEZ-4144
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: László Bodor
>            Priority: Trivial
>
> Precommit checkstyle result always complains about this one:
> {code}
> ./tez-tools/analyzers/job-analyzer/src/main/java/org/apache/tez/analyzer/plugins/TaskAttemptResultStatisticsAnalyzer.java:46:      { "vertex (+task stats: all/succeeded/failed/killed)", "node", "status", "numAttempts" };:7: '{' is followed by whitespace. [NoWhitespaceAfter]
> {code}
> is really an issue?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)