Gabriel39 opened a new pull request, #69119:
URL: https://github.com/apache/doris/pull/69119
### What problem does this PR solve?
Related PRs: #69092, #69094
Port the Parquet V2 diagnostic improvement to branch-4.2. Footer/schema
failures currently omit the input file path, and Variant fallback errors
identify only a wrapper name such as `element`.
Attach the file path 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; accepted layouts and read semantics remain
unchanged.
### Validation
- Clean cherry-pick of #69092. All three affected files match the source PR
exactly, and the patch IDs are identical.
- clang-format 16.0.6 verification on all affected files and `git diff
--check` passed.
- The source PR passed all 45 `ParquetSchemaTest.*` tests in a focused ASAN
binary on branch-4.1. No new local build or unit-test run was performed for
this branch-4.2 port; branch validation is pending CI.
- Includes three diagnostic tests covering required/repeated fallback
leaves, incorrect physical types, group fallback values, a valid control,
unsupported Variant versions, and truncated footers.
### 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; branch-4.2 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]
