Cleaning up more comments and one more test.

Author: culler

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/types/timetypes.scala

@@ -20,7 +20,7 @@ package org.apache.spark.sql.catalyst.expressions
 import java.sql.{Date, Timestamp}
 import scala.language.implicitConversions
 
-/* *
+/**
  * Subclass of java.sql.Date which provides the usual comparison
  * operators (as required for catalyst expressions) and which can
  * be constructed from a string.

sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala

@@ -503,7 +503,7 @@ class SQLContext(@transient val sparkContext: SparkContext)
     new SchemaRDD(this, LogicalRDD(schema.toAttributes, rowRdd)(self))
   }
 
-  /* *
+  /**
    * Make RichDate and RichTimestamp available under the names
    * Date and Timestamp when the members of this SQLContext are
    * imported.

sql/hive/src/test/scala/org/apache/spark/sql/parquet/HiveParquetSuite.scala

@@ -29,13 +29,6 @@ import org.apache.spark.util.Utils
 // Implicits
 import org.apache.spark.sql.hive.test.TestHive._
 
-/*
- * Note: the DSL conversions collide with the scalatest === operator!
- * We can apply the scalatest conversion explicitly:
- *   assert(X === Y) --> assert(convertToEqualizer(X).===(Y))
- * (This file already imports convertToEqualizer)
- */
-
 case class Cases(lower: String, UPPER: String)
 
 class HiveParquetSuite extends FunSuite with BeforeAndAfterAll with BeforeAndAfterEach {
@@ -87,7 +80,7 @@ class HiveParquetSuite extends FunSuite with BeforeAndAfterAll with BeforeAndAft
 
   test("Simple column projection + filter on Parquet table") {
     val rdd = sql("SELECT myboolean, mylong FROM testsource WHERE myboolean=true").collect()
-    assert(convertToEqualizer(rdd.size).===(5), "Filter returned incorrect number of rows")
+    assert(rdd.size === 5, "Filter returned incorrect number of rows")
     assert(rdd.forall(_.getBoolean(0)), "Filter returned incorrect Boolean field value")
   }
 
@@ -109,7 +102,7 @@ class HiveParquetSuite extends FunSuite with BeforeAndAfterAll with BeforeAndAft
     sql("INSERT OVERWRITE TABLE ptable SELECT * FROM testsource").collect()
     val rddCopy = sql("SELECT * FROM ptable").collect()
     val rddOrig = sql("SELECT * FROM testsource").collect()
-    assert(convertToEqualizer(rddCopy.size).===(rddOrig.size), "INSERT OVERWRITE changed size of table??")
+    assert(rddCopy.size === rddOrig.size, "INSERT OVERWRITE changed size of table??")
     compareRDDs(rddOrig, rddCopy, "testsource", ParquetTestData.testSchemaFieldNames)
   }
 
@@ -118,8 +111,7 @@ class HiveParquetSuite extends FunSuite with BeforeAndAfterAll with BeforeAndAft
     (rddOne, rddTwo).zipped.foreach {
       (a,b) => (a,b).zipped.toArray.zipWithIndex.foreach {
         case ((value_1, value_2), index) =>
-          assert(convertToEqualizer(value_1).===(value_2),
-		 s"table $tableName row $counter field ${fieldNames(index)} don't match")
+          assert(value_1 === value_2, s"table $tableName row $counter field ${fieldNames(index)} don't match")
       }
     counter = counter + 1
     }