You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "sarutak (via GitHub)" <gi...@apache.org> on 2023/09/04 08:06:07 UTC

[GitHub] [avro] sarutak opened a new pull request, #2481: AVRO-3851: [Rust] Validate default value for record fields and enum on parsing

sarutak opened a new pull request, #2481:
URL: https://github.com/apache/avro/pull/2481

   AVRO-3851
   
   ## What is the purpose of the change
   This PR proposes to improve the functionality of parsing default values for record fields and enums.
   
   Currently, default values for record fields are validated on parsing only if their types are `union`.
   It's nice to apply such validation for other types.
   Similarly, default values for `enum` are not also validated on parsing now.
   
   So this PR changes parser functionality to validate default values for record fields and enums.
   
   ## Verifying this change
   Added new tests.
   
   ## Documentation
   
   - Does this pull request introduce a new feature? (no)
   


-- 
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: dev-unsubscribe@avro.apache.org

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

[GitHub] [avro] martin-g commented on pull request #2481: AVRO-3851: [Rust] Validate default value for record fields and enums on parsing

Posted by "martin-g (via GitHub)" <gi...@apache.org>.
martin-g commented on PR #2481:
URL: https://github.com/apache/avro/pull/2481#issuecomment-1710294930

   Thank you, @sarutak !
   Great work !


-- 
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: issues-unsubscribe@avro.apache.org

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

[GitHub] [avro] martin-g merged pull request #2481: AVRO-3851: [Rust] Validate default value for record fields and enums on parsing

Posted by "martin-g (via GitHub)" <gi...@apache.org>.
martin-g merged PR #2481:
URL: https://github.com/apache/avro/pull/2481


-- 
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: dev-unsubscribe@avro.apache.org

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