chamikaramj commented on issue #36496:
URL: https://github.com/apache/beam/issues/36496#issuecomment-3402626760

   The Row is generated from the old Beam version and used to create 
schema-aware transforms in new Beam versions. I think given that we sort fields 
at all locations and ignore unknown fields, this end up being working but we 
might run into issues when modifying the row and and trying to use it unsorted. 
Also, @reuvenlax mentioned elsewhere that sorting field is incompatible with 
some of the update compatibility related work that is being worked on in the 
Dataflow service side.
   
   @ahmedabu98 I think we added the sorting since the schema generated by 
``TypedSchemaTransformProvider`` was unstable. Will we be able to remove the 
sorting by adding ``SchemaFieldNumber`` annotations to all schema-aware 
transforms similar to 
[https://github.com/apache/beam/pull/36295](https://github.com/apache/beam/issues/36496#top)
 ?


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to