Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21404#discussion_r190097176
  
    --- Diff: 
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
    @@ -1148,3 +1149,128 @@ private[client] class Shim_v2_1 extends Shim_v2_0 {
     private[client] class Shim_v2_2 extends Shim_v2_1
     
     private[client] class Shim_v2_3 extends Shim_v2_1
    +
    +private[client] class Shim_v3_0 extends Shim_v2_3 {
    +  // Spark supports only non-ACID operations
    +  protected lazy val isAcidIUDoperation = JBoolean.FALSE
    +
    +  // Writer ID can be 0 for non-ACID operations
    +  protected lazy val writeIdInLoadTableOrPartition: JLong = 0L
    +
    +  // Statement ID
    +  protected lazy val stmtIdInLoadTableOrPartition: JInteger = 0
    +
    +  protected lazy val listBucketingLevel: JInteger = 0
    +
    +  private lazy val clazzLoadFileType = getClass.getClassLoader.loadClass(
    +    "org.apache.hadoop.hive.ql.plan.LoadTableDesc$LoadFileType")
    +
    +  private lazy val loadPartitionMethod =
    --- End diff --
    
    BTW, I tracked and checked all the signature changed.


---

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

Reply via email to