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]