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

    https://github.com/apache/spark/pull/19083#discussion_r142559152
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/WholeStageCodegenSuite.scala
 ---
    @@ -151,7 +151,7 @@ class WholeStageCodegenSuite extends SparkPlanTest with 
SharedSQLContext {
         }
       }
     
    -  def genGroupByCodeGenContext(caseNum: Int): CodegenContext = {
    +  def genGroupByCodeGenContext(caseNum: Int): (CodegenContext, 
CodeAndComment) = {
    --- End diff --
    
    The returned `CodegenContext` is not used. We don't need to return it.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to