linkedin / rest.li

Rest.li is a REST+JSON framework for building robust, scalable service architectures using dynamic discovery and simple asynchronous APIs.
rest.li
Other
2.51k stars 546 forks source link

Translate optional map/arrays with defaults to Avro without breaking #955

Closed dg-builder closed 11 months ago

dg-builder commented 11 months ago

This PR changes the translation of default values for optional fields when TRANSLATE_DEFAULT is enabled. Before, there were cases (especially for nested records) where non-null default values would not be translated. This change enables translating default values (excluding unions, records, and typerefs).

dg-builder commented 11 months ago

closing for now. will reopen a broader solution later.