Simplify filter

petermaxlee · petermaxlee · commit 749774261742 · 2016-08-09T23:04:38.000-07:00
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala
@@ -124,7 +124,7 @@ class SQLQueryTestSuite extends QueryTest with SharedSQLContext {
       // val cleaned = input.split("\n").filterNot(_.matches("--.*(?<=[^\\\\]);")).mkString("\n")
       val cleaned = input.split("\n").filterNot(_.startsWith("--")).mkString("\n")
       // note: this is not a robust way to split queries using semicolon, but works for now.
-      cleaned.split("(?<=[^\\\\]);").map(_.trim).filterNot(q => q == "").toSeq
+      cleaned.split("(?<=[^\\\\]);").map(_.trim).filter(_ != "").toSeq
     }
 
     // Run the SQL queries preparing them for comparison.

Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ class SQLQueryTestSuite extends QueryTest with SharedSQLContext {`
`124`	`124`	`// val cleaned = input.split("\n").filterNot(_.matches("--.*(?<=[^\\\\]);")).mkString("\n")`
`125`	`125`	`val cleaned = input.split("\n").filterNot(_.startsWith("--")).mkString("\n")`
`126`	`126`	`// note: this is not a robust way to split queries using semicolon, but works for now.`
`127`		`- cleaned.split("(?<=[^\\\\]);").map(_.trim).filterNot(q => q == "").toSeq`
	`127`	`+ cleaned.split("(?<=[^\\\\]);").map(_.trim).filter(_ != "").toSeq`
`128`	`128`	`}`
`129`	`129`
`130`	`130`	`// Run the SQL queries preparing them for comparison.`