[GitHub] [beam] chamikaramj commented on issue #25669: [Bug]: Configuration row arguments may get misplaced between Python SchemaTransformPayload encoding and Java RowCoder decoding

["chamikaramj (via GitHub)" <gi...@apache.org>]

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

   jdbc.py has a similar issue but that's not addressed here. transforms in jdbc.py needs to preserve ordering since it uses a second config object [1] along with (old) SchemaIO API. This config object that is encoded in the Python side and decoded in the Java side. So ordering of the the objects in the two sides have to match.
   
   One way to fix this will be to update jdbc.py to use schema-aware transforms (which will pick up the fix provided in this PR).
   
   [1] https://github.com/apache/beam/blob/e868c8da79a8cf727044cf805b2041fbdb8ff181/sdks/python/apache_beam/io/jdbc.py#L339
   


-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org