csun5285 opened a new pull request, #64011:
URL: https://github.com/apache/doris/pull/64011

   Stream Load into a STRUCT column reads each value as a string and converts 
it with DataTypeStructSerDe::from_string (CAST varchar -> struct). That path 
matched sub-fields by position, so JSON keys whose order differed from the DDL 
turned the whole struct column into NULL, and a row missing a field failed to 
load.
   
   from_string now detects named mode by the delimiter structure and matches 
sub-fields by name (case-insensitive), fills missing nullable fields with NULL, 
and ignores unknown/extra fields. This is consistent with the simdjson JSON 
reader and with PostgreSQL / Spark / Trino. Positional input still requires an 
exact field count, and struct-to-struct CAST stays positional, which also 
matches those engines.
   
   Add BE unit tests (DataTypeStructSerDeTest.FromStringByFieldName) covering 
by-name matching, case-insensitivity, missing/unknown/extra fields, positional 
input and the error paths; and a stream-load regression test 
(test_struct_field_align).
   
   ### What problem does this PR solve?
   
   Issue Number: close #xxx
   
   Related PR: #xxx
   
   Problem Summary:
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test <!-- At least one of them must be included. -->
       - [x] Regression test
       - [x] Unit Test
       - [ ] Manual test (add detailed scripts or steps below)
       - [ ] No need to test or manual test. Explain why:
           - [ ] This is a refactor/code format and no logic has been changed.
           - [ ] Previous test can cover this change.
           - [ ] No code files have been changed.
           - [ ] Other reason <!-- Add your reason?  -->
   
   - Behavior changed:
       - [ ] No.
       - [ ] Yes. <!-- Explain the behavior change -->
   
   - Does this need documentation?
       - [ ] No.
       - [ ] Yes. <!-- Add document PR link here. eg: 
https://github.com/apache/doris-website/pull/1214 -->
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label <!-- Add branch pick label that this PR should 
merge into -->
   
   


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