You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Reynold Xin (JIRA)" <ji...@apache.org> on 2015/05/31 07:03:19 UTC

[jira] [Created] (SPARK-7977) Disallow println

Reynold Xin created SPARK-7977:
----------------------------------

             Summary: Disallow println
                 Key: SPARK-7977
                 URL: https://issues.apache.org/jira/browse/SPARK-7977
             Project: Spark
          Issue Type: Sub-task
            Reporter: Reynold Xin


Very often we see pull requests that added println from debugging, but the author forgot to remove it before code review.

We can use the regex checker to disallow println. For legitimate use of println, we can then disable the rule.

Add to scalastyle-config.xml file:
{code}
  <check customId="println" level="error" class="org.scalastyle.scalariform.TokenChecker" enabled="true">
    <parameters><parameter name="regex">^println$</parameter></parameters>
    <customMessage><![CDATA[Are you sure you want to println? If yes, wrap the code block with 
      // scalastyle:off println
      println(...)
      // scalastyle:on println]]></customMessage>
  </check>
{code}




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

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org