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

   AVRO-3827
   
   ## What is the purpose of the change
   
   This PR aims to fix an issue that the current Rust binding accepts duplicate 
field names.
   If a schema contains a record and some of its fields have the same field 
name, such schema should not be allowed.
   ```
   {
     "name": "my_schema",
     "type": "record",
     "fields": [
       {
         "name": "f1",
         "type": {
           "name": "a",
           "type": "record",
           "fields": []
         }
       },  {
         "name": "f1",
         "type": {
           "name": "b",
           "type": "record",
           "fields": []
         }
       }
     ]
    }
   ```
   
   ## Verifying this change
   Added new test and passed with `cargo test avro_3827`.
   
   This change added tests and can be verified as follows:
   
   *(example:)*
   - *Extended interop tests to verify consistent valid schema names between 
SDKs*
   - *Added test that validates that Java throws an AvroRuntimeException on 
invalid binary data*
   - *Manually verified the change by building the website and checking the new 
redirect*
   
   ## Documentation
   No new features added.


-- 
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-unsubscr...@avro.apache.org

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

Reply via email to