You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Weiqing Yang (JIRA)" <ji...@apache.org> on 2016/11/21 07:27:59 UTC

[jira] [Created] (SPARK-18521) Add `NoRedundantStringInterpolator` Scala rule

Weiqing Yang created SPARK-18521:
------------------------------------

             Summary: Add `NoRedundantStringInterpolator` Scala rule
                 Key: SPARK-18521
                 URL: https://issues.apache.org/jira/browse/SPARK-18521
             Project: Spark
          Issue Type: Improvement
            Reporter: Weiqing Yang


Currently the s string interpolator is used in many cases in which there is no embed variable reference in the processed string literals. 
For example:
core/src/main/scala/org/apache/spark/deploy/Client.scala
{code}
logError(s"Error processing messages, exiting.")
{code}

examples/src/main/scala/org/apache/spark/examples/graphx/SynthBenchmark.scala
{code}
println(s"Creating graph...")
{code}

examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala
{code}
println(s"Corpus summary:")
{code}

sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala
{code}
test(s"correctly handle CREATE OR REPLACE TEMPORARY VIEW") {
{code}

We can add a new scala style rule 'NoRedundantStringInterpolator' to prevent unnecessary string interpolators. 



--
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