[jira] [Updated] (SPARK-7977) Disallow println
[ https://issues.apache.org/jira/browse/SPARK-7977?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Wendell updated SPARK-7977: --- Labels: starter (was: spark.tc starter) > Disallow println > > > Key: SPARK-7977 > URL: https://issues.apache.org/jira/browse/SPARK-7977 > Project: Spark > Issue Type: Sub-task > Components: Project Infra >Reporter: Reynold Xin >Assignee: Jon Alter > Labels: starter > Fix For: 1.5.0 > > > 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 where they are used. > Add to scalastyle-config.xml file: > {code} >class="org.scalastyle.scalariform.TokenChecker" enabled="true"> > ^println$ > > > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Updated] (SPARK-7977) Disallow println
[ https://issues.apache.org/jira/browse/SPARK-7977?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jon Alter updated SPARK-7977: - Labels: spark.tc starter (was: starter) > Disallow println > > > Key: SPARK-7977 > URL: https://issues.apache.org/jira/browse/SPARK-7977 > Project: Spark > Issue Type: Sub-task > Components: Project Infra >Reporter: Reynold Xin >Assignee: Jon Alter > Labels: spark.tc, starter > Fix For: 1.5.0 > > > 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 where they are used. > Add to scalastyle-config.xml file: > {code} >class="org.scalastyle.scalariform.TokenChecker" enabled="true"> > ^println$ > > > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Updated] (SPARK-7977) Disallow println
[ https://issues.apache.org/jira/browse/SPARK-7977?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sean Owen updated SPARK-7977: - Assignee: Jon Alter > Disallow println > > > Key: SPARK-7977 > URL: https://issues.apache.org/jira/browse/SPARK-7977 > Project: Spark > Issue Type: Sub-task > Components: Project Infra >Reporter: Reynold Xin >Assignee: Jon Alter > Labels: starter > Fix For: 1.5.0 > > > 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 where they are used. > Add to scalastyle-config.xml file: > {code} >class="org.scalastyle.scalariform.TokenChecker" enabled="true"> > ^println$ > > > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Updated] (SPARK-7977) Disallow println
[ https://issues.apache.org/jira/browse/SPARK-7977?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Reynold Xin updated SPARK-7977: --- Description: 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 where they are used. Add to scalastyle-config.xml file: {code} ^println$ {code} was: 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} ^println$ {code} > Disallow println > > > Key: SPARK-7977 > URL: https://issues.apache.org/jira/browse/SPARK-7977 > Project: Spark > Issue Type: Sub-task > Components: Project Infra >Reporter: Reynold Xin > Labels: starter > > 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 where they are used. > Add to scalastyle-config.xml file: > {code} >class="org.scalastyle.scalariform.TokenChecker" enabled="true"> > ^println$ > > > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Updated] (SPARK-7977) Disallow println
[ https://issues.apache.org/jira/browse/SPARK-7977?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Reynold Xin updated SPARK-7977: --- Labels: starter (was: ) > Disallow println > > > Key: SPARK-7977 > URL: https://issues.apache.org/jira/browse/SPARK-7977 > Project: Spark > Issue Type: Sub-task > Components: Project Infra >Reporter: Reynold Xin > Labels: starter > > 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} >class="org.scalastyle.scalariform.TokenChecker" enabled="true"> > ^println$ > > > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org