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/04 09:31:43 UTC

[GitHub] [beam] kanterov edited a comment on issue #11034: [BEAM-9424] Allow grouping by LogicalType

kanterov edited a comment on issue #11034: [BEAM-9424] Allow grouping by LogicalType
URL: https://github.com/apache/beam/pull/11034#issuecomment-594414157
 
 
   @reuvenlax because `RowCoderGenerator` uses `Coder<BaseT>`, not `Coder<InputT>`. We know how to produce `Coder<BaseT>`, and we can make `Coder<InputT>` out of it using `LogicalType`. So the test fails because Row builder expected `EnumerationType.Value` but got `String`. What is special about Group is that it constructs coders for keys and values.

----------------------------------------------------------------
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