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:
[email protected]


With regards,
Apache Git Services

Reply via email to