[GitHub] [flink] bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition

2019-07-10 Thread GitBox
bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition URL: https://github.com/apache/flink/pull/9004#issuecomment-510250433 merged in 1.9.0:

[GitHub] [flink] bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition

2019-07-10 Thread GitBox
bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition URL: https://github.com/apache/flink/pull/9004#issuecomment-510234607 @xuefuz thanks, I'll merge it first since it will

[GitHub] [flink] bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition

2019-07-08 Thread GitBox
bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition URL: https://github.com/apache/flink/pull/9004#issuecomment-509358595 > The changes look good to me. However, I don't know

[GitHub] [flink] bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition

2019-07-08 Thread GitBox
bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition URL: https://github.com/apache/flink/pull/9004#issuecomment-509336235 cc @xuefuz

[GitHub] [flink] bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition

2019-07-05 Thread GitBox
bowenli86 commented on issue #9004: [FLINK-13128][hive] make HiveGenericUDAF expose accumulator type in order to create its corresponding AggregateFunctionDefinition URL: https://github.com/apache/flink/pull/9004#issuecomment-508874495 cc @xuefuz @JingsongLi @lirui-apache @zjuwangg