jackjlli opened a new pull request #6009:
URL: https://github.com/apache/incubator-pinot/pull/6009


   ## Description
   This PR adjusts schema validation logic in AvroIngestionSchemaValidator.
   
   The current logic doesn't check the actual data type for multi-value column, 
which could have returned incorrect validation results.
   
   E.g. if column3 is of array structure (like Object[]) and its base element 
is of String type in AVRO and if column3 is of String type as well in Pinot 
schema, the current code would mark it `data type mismatch`.
   
   E.g. if column3 is of array of map structure (like Map[]), the current logic 
would miss marking it `multi-value structure mismatch`.


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to