rahul8383 commented on pull request #11609:
URL: https://github.com/apache/beam/pull/11609#issuecomment-624906438


   > This seems to be a holdover. Previously Row stored logical type values as 
their base type, so we probably called `toBaseType(toInputType(x))`.
   
   Even before the code which changed `Row` to store logical type values 
instead of base values in memory, while building the row, the input value is 
expected to be of correct length.
   
https://github.com/apache/beam/blob/9f0cb649d39ee6236ea27f111acb4b66591a80ec/sdks/java/core/src/main/java/org/apache/beam/sdk/values/Row.java#L658-L660
   I think this issue is present since the introduction of `FixedBytes`.


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


Reply via email to