LuciferYang opened a new pull request, #9767:
URL: https://github.com/apache/paimon/pull/9767

   ### Purpose
   
   close #9766
   
   The public `DataTypeJsonParser.parseDataField(JsonNode)` entry passed a null 
field counter, so a field json without an `"id"` node threw a bare NPE — while 
the sibling public `parseDataType` entry auto-assigns ids for exactly such 
input. This mirrors that behavior: a fresh counter is passed, missing ids 
auto-assign from 0, explicit ids keep working, and the private overload's 
partial-id `checkState` is unchanged.
   
   Reachable from the `create function` procedures via 
`ParameterUtils.parseDataFieldArray` on user-provided parameter json.
   
   ### Tests
   
   New `DataTypeJsonParserTest`: id-less field auto-assigns 0 (RED on master — 
NPE), explicit id preserved, nested ROW auto-assigns sequentially.
   
   ### API and Format
   
   No behavior change for previously-working inputs (all-explicit-id json never 
touches the counter). Previously-crashing id-less json now parses with an 
auto-assigned id.
   
   ### Documentation
   
   None.


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

Reply via email to