jdarais commented on PR #139:
URL: https://github.com/apache/avro-rs/pull/139#issuecomment-2705686643

   > @jdarais I just pushed a failing test with a recursive RecordSchema. Could 
you please check what is the problem ? Thanks!
   > 
   > Update: I think I fixed it with 
[a364ac9](https://github.com/apache/avro-rs/commit/a364ac90a4866625189af8e87fb1b9de829e5929)
   
   Is that recursive schema a legal one, (or at least able to be non-infinite)? 
 For the recursive `innerRecord` field in the `TestRecord` schema, I think it 
would need to be `{"name": "innerRecord", "type": ["null", "TestRecord"]}` in 
order to accommodate the terminal record that has `None` for its `innerRecord` 
field.  With that modification to the schema, the object created in 
`test_serialize_recursive_record` should be able to be serialized without 
errors.


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