Gabriel39 opened a new pull request, #69094:
URL: https://github.com/apache/doris/pull/69094
### What problem does this PR solve?
Related PR: #69092
Parquet V2 footer/schema errors can reach the client without identifying the
input file. Invalid Variant fallback errors report only a wrapper name such as
`element`, making nested failures difficult to locate.
Include the source file path in native footer errors while preserving the
original status code and message. Invalid Variant fallback errors also include
the Variant-relative field path, declared physical type, repetition, and child
count. Diagnostic paths are constructed only on failure; normalized repeated
primitives report their original declarations.
This ports #69092 to master, retaining master's metadata ownership and
`preserve_binary_uuid` argument. Accepted file layouts and read semantics are
unchanged.
### Validation
- Repository clang-format 16 check, build hygiene checks, and `git diff
--check` passed.
- Ported the three diagnostic unit tests, covering required/repeated
fallback leaves, incorrect physical types, group fallback values, a valid
control, unsupported Variant versions, and truncated footers.
- The source PR passed all 45 `ParquetSchemaTest.*` tests in a focused ASAN
binary on branch-4.1. The master port has not been compiled or unit-tested
locally; master build/test validation is pending CI.
### Release note
Parquet V2 footer errors now identify the input file. Invalid Variant
fallback errors additionally describe the field path and actual schema
declaration.
### Check List (For Author)
- Test
- [x] Unit Test (ported; master execution pending CI)
- Behavior changed:
- [x] Yes. Diagnostic context is added; status codes and validation
rules are preserved.
- Does this need documentation?
- [x] No.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
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]
