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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to