You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by GitBox <gi...@apache.org> on 2020/03/03 23:07:36 UTC
[GitHub] [beam] kanterov commented on issue #11034: [BEAM-9424] Allow
grouping by LogicalType
kanterov commented on issue #11034: [BEAM-9424] Allow grouping by LogicalType
URL: https://github.com/apache/beam/pull/11034#issuecomment-594217964
@reuvenlax it looks like there is a couple of places where LogicalType don't work since we introduced to/fromBaseType functions. Looking into the code we would need to support it in RowCoderGenerator that doesn't look straightforward because LogicalType doesn't have a static factory we can call.
Do you have thoughts on how we can fix that?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services