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

   ### What problem does this PR solve?
   
   Issue Number: N/A
   
   Related PR: N/A
   
   Problem Summary: Legacy Iceberg Parquet and ORC files without field IDs are 
resolved through name mapping. With a partial mapping, FE omitted the optional 
mapping metadata for unmapped fields, so both V1 and V2 BE readers fell back to 
the current physical name and read unrelated data.
   
   This change preserves table-level mapping presence by transporting explicit 
empty per-field lists and makes those lists authoritative in both readers. 
Unmapped fields now materialize their default or NULL, while mapped aliases and 
scans without name mapping retain their existing behavior.
   
   ### Release note
   
   Fix reads of migrated Iceberg files with partial name mapping so fields 
omitted from the mapping materialize their default value or NULL instead of 
matching a physical column by the current name.
   
   ### Check List (For Author)
   
   - Test: Unit Test
       - `ExternalUtilTest` (6 tests)
       - Focused V1 and V2 name-mapping BE unit tests under ASAN (10 tests)
   - Behavior changed: Yes. Partial Iceberg name mappings are now authoritative 
for legacy files without field IDs.
   - Does this need documentation: No
   


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