martin-g commented on PR #260:
URL: https://github.com/apache/avro-rs/pull/260#issuecomment-3178753923
Hm!
I think there is a bug here.
Currently this produces:
```
RecordField {
...,
default: Some(
String("null"),
),
...
```
while it should be `default: Some(Value::Null)`.
I see two ways to improve/fix it:
1) Update our `darling` usage to support `#[avro(default = null)]`, notice
that `null` is not a String!
2) If 1) does not work for some reason then check whether the value is
`"null"` and the first union variant is `Null` then use `Value::Null`
--
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]
