Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19882#discussion_r155392637
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcTest.scala
 ---
    @@ -15,20 +15,51 @@
      * limitations under the License.
      */
     
    -package org.apache.spark.sql.hive.orc
    +package org.apache.spark.sql.execution.datasources.orc
     
     import java.io.File
     
     import scala.reflect.ClassTag
     import scala.reflect.runtime.universe.TypeTag
     
    +import org.scalatest.BeforeAndAfterAll
    +
     import org.apache.spark.sql._
    -import org.apache.spark.sql.hive.test.TestHiveSingleton
    +import org.apache.spark.sql.internal.SQLConf.ORC_IMPLEMENTATION
     import org.apache.spark.sql.test.SQLTestUtils
     
    -private[sql] trait OrcTest extends SQLTestUtils with TestHiveSingleton {
    +/**
    + * OrcTest
    + *   -> OrcSuite
    + *       -> OrcSourceSuite
    + *       -> HiveOrcSourceSuite
    + *   -> OrcQueryTests
    + *       -> OrcQuerySuite
    + *       -> HiveOrcQuerySuite
    + *   -> OrcPartitionDiscoveryTest
    + *       -> OrcPartitionDiscoverySuite
    + *       -> HiveOrcPartitionDiscoverySuite
    + *   -> OrcFilterSuite
    + *   -> HiveOrcFilterSuite
    + */
    +abstract class OrcTest extends QueryTest with SQLTestUtils with 
BeforeAndAfterAll {
       import testImplicits._
     
    +  def orcImp: String = "native"
    --- End diff --
    
    Yep. Done.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to