sbernauer commented on issue #1845:
URL: https://github.com/apache/hudi/issues/1845#issuecomment-660853366


   Thanks for your fast reply!
   
   The PR adds a new Test and improves 2 existing tests. The mentioned 4 new 
cols in TestHoodieAvroUtils increase the number of tested cases, the tests are 
still working.
   The new test is in TestHoodieDeltaStreamer and starts the DeltaStreamer with 
different schemas and transformers to reproduce the actual problem.
   
   Regarding the both versions of my schema, i can only provide a diff but it 
should be sufficient. The new field has the same type (union of null and 
string, default of null) as reproduced in the test here 
https://github.com/apache/hudi/pull/1844/files#diff-07dd5ed6077721a382c35b2700da0883R130.
   
   diff Event.json_schema Event-aged.json_schema 
   >     },
   >     {
   >       "name": "agedOptionalField",
   >       "type": [
   >         "null",
   >         {
   >           "type": "string",
   >           "avro.java.string": "String"
   >         }
   >       ],
   >       "doc": "New aged optional field",
   >       "default": 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.

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


Reply via email to