[GitHub] flink pull request #5940: [FLINK-8690][table]Support group window distinct a...

[suez1224]

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

    https://github.com/apache/flink/pull/5940#discussion_r184866214
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/logical/FlinkLogicalAggregate.scala
 ---
    @@ -82,7 +82,7 @@ private class FlinkLogicalAggregateConverter
           case _ => true
         }
     
    -    !agg.containsDistinctCall() && supported
    +    supported
    --- End diff --
    
    I dont think we need this extra local variable here after your change.


---