[GitHub] [beam] robinyqiu commented on pull request #12120: [BEAM-10224] Test group by and aggregation on DATE and TIME type

2020-06-29 Thread GitBox
robinyqiu commented on pull request #12120: URL: https://github.com/apache/beam/pull/12120#issuecomment-651426652 The new ZetaSQL `DATE` and `TIME` types use Beam logical types. Currently Beam SQL transforms operate on the **base types** of logical types. This produces incorrect result

[GitHub] [beam] robinyqiu commented on pull request #12120: [BEAM-10224] Test group by and aggregation on DATE and TIME type

2020-06-29 Thread GitBox
robinyqiu commented on pull request #12120: URL: https://github.com/apache/beam/pull/12120#issuecomment-651419850 retest this please This is an automated message from the Apache Git Service. To respond to the message, please