Abacn commented on code in PR #34542:
URL: https://github.com/apache/beam/pull/34542#discussion_r2028917880


##########
sdks/python/apache_beam/io/avroio.py:
##########
@@ -646,9 +646,9 @@ def avro_atomic_value_to_beam_atomic_value(avro_type: str, 
value):
   if value is None:
     return value
   elif avro_type == "int":
-    return ctypes.c_uint32(value).value

Review Comment:
   We may simply revert the workaround in #30770 that is remove this helper 
function, and see if 
`test_avro_schema_to_beam_schema_with_nullable_atomic_fields` still pass



-- 
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-unsubscr...@beam.apache.org

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

Reply via email to