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

    https://github.com/apache/spark/pull/19363#discussion_r143094589
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/KeyValueGroupedDataset.scala ---
    @@ -18,16 +18,17 @@
     package org.apache.spark.sql
     
     import scala.collection.JavaConverters._
    +import scala.util.control.NonFatal
     
     import org.apache.spark.annotation.{Experimental, InterfaceStability}
     import org.apache.spark.api.java.function._
     import org.apache.spark.sql.catalyst.encoders.{encoderFor, 
ExpressionEncoder}
     import org.apache.spark.sql.catalyst.expressions.{Alias, Attribute, 
CreateStruct}
     import org.apache.spark.sql.catalyst.plans.logical._
    -import org.apache.spark.sql.catalyst.streaming.InternalOutputModes
     import org.apache.spark.sql.execution.QueryExecution
     import org.apache.spark.sql.expressions.ReduceAggregator
     import org.apache.spark.sql.streaming.{GroupState, GroupStateTimeout, 
OutputMode}
    +import org.apache.spark.sql.types.StructType
    --- End diff --
    
    Why import `StructType`? I didn't see you use it.


---

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

Reply via email to