Details
-
Bug
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
None
Description
The Name struct derives the trait Deserialize, which allows it to be constructed from a JSON as shown in this test: https://github.com/apache/avro/blob/master/lang/rust/avro/tests/schema.rs#L1080
(called test_fullname_fullname_and_namespace_specified())
However, the derive flow does not parse out the namespace from the name which is handled correctly from both Name::new() and Name:: parse() , which leads to a struct this is invalid against what is expected.
It would be nice to remove the derive Deserialize trait, and replace with a custom implementation or just removing the implementation all together (and double checking for side effects of that)
Attachments
Issue Links
- links to