address comments.

Author: gatorsmile

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

@@ -21,7 +21,6 @@ import java.util.Properties
 
 import scala.collection.JavaConverters._
 
-import org.apache.spark.Logging
 import org.apache.spark.annotation.Experimental
 import org.apache.spark.sql.catalyst.{CatalystQl, TableIdentifier}
 import org.apache.spark.sql.catalyst.analysis.UnresolvedRelation
@@ -38,7 +37,7 @@ import org.apache.spark.sql.sources.HadoopFsRelation
  * @since 1.4.0
  */
 @Experimental
-final class DataFrameWriter private[sql](df: DataFrame) extends Logging  {
+final class DataFrameWriter private[sql](df: DataFrame) {
 
   /**
    * Specifies the behavior when data or table already exists. Options include:

sql/hive/src/test/scala/org/apache/spark/sql/hive/MetastoreDataSourcesSuite.scala

@@ -778,7 +778,7 @@ class MetastoreDataSourcesSuite extends QueryTest with SQLTestUtils with TestHiv
 
   test("Saving information for sortBy and bucketBy columns") {
     val df = (1 to 10).map(i => (i, i + 1, s"str$i", s"str${i + 1}")).toDF("a", "b", "c", "d")
-    val tableName = s"partitionInfo_${System.currentTimeMillis()}"
+    val tableName = s"bucketingInfo_${System.currentTimeMillis()}"
 
     withTable(tableName) {
       df.write

sql/hive/src/test/scala/org/apache/spark/sql/sources/BucketedWriteSuite.scala

@@ -163,8 +163,8 @@ class BucketedWriteSuite extends QueryTest with SQLTestUtils with TestHiveSingle
 
   test("write bucketed data with the overlapping bucketBy and partitionBy columns") {
     intercept[AnalysisException](df.write
-      .partitionBy("i")
-      .bucketBy(8, "i", "k")
+      .partitionBy("i", "j")
+      .bucketBy(8, "j", "k")
       .sortBy("k")
       .saveAsTable("bucketed_table"))
   }