clintropolis opened a new pull request, #19769:
URL: https://github.com/apache/druid/pull/19769

   ### Description
   This PR improves catalog type validation to reject invalid complex types 
instead of silently being ignored. It also updates druid native type validation 
to perform case insensitive handling of `ARRAY` and `COMPLEX` type prefixes, to 
be more consistent with how primitive types are handled (case insensitive), and 
fixes bug with handling of malformed type strings without a closing `>`. Prior 
to this something like `long` was allowed, but `complex<json>` or `array<long>` 
was not allowed (while `ARRAY<long>` was since array parsing recurses after 
removing the prefix). Array and complex prefix handling is now case 
insensitive, but complex type name handling itself must still be exact, as the 
underlying complex type registry is still case sensitive.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to