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

    https://github.com/apache/flink/pull/4873#discussion_r147120015
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/AggregateUtil.scala
 ---
    @@ -849,17 +848,7 @@ object AggregateUtil {
           outputType
         )
     
    -    val constantFlags: Option[Array[(Int, Boolean)]] =
    -    if (inGroupingSet) {
    -
    -      val groupingSetsMapping = getGroupingSetsIndicatorMapping(inputType, 
outputType)
    --- End diff --
    
    `getGroupingSetsIndicatorMapping()` method can be removed as well 
everything related to the `constantFlags` parameter in 
`AggregationCodeGenerator`, `GeneratedAggregations`, 
`DawtaSetFinalAggFunction`, etc.


---

Reply via email to