Gabriel39 opened a new pull request, #66408:
URL: https://github.com/apache/doris/pull/66408
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary:
Paimon system table safety validation wrapped actionable reader-option
errors with a generic connector message. As a result, SQL clients could not
identify the invalid option even though the original cause contained it.
This change preserves `IllegalArgumentException` validation details at the
SQL boundary while retaining the original cause. Other unexpected failures
continue to use the generic connector message.
### Release note
Paimon system table validation errors now report the invalid reader option.
### Check List (For Author)
- Test
- [ ] Regression test
- [x] Unit Test
- `mvn -f fe/pom.xml -pl :fe-connector-paimon -am test
-Dtest=PaimonScanPlanProviderTest#resolveSystemScanTableValidatesHiddenDataTable
-DfailIfNoTests=false`
- `mvn -f fe/pom.xml -pl :fe-connector-paimon -am package
-Dmaven.build.cache.enabled=false -DfailIfNoTests=false`
- Paimon module: 505 tests, 0 failures, 1 skipped live-connectivity
test.
- FE Checkstyle: 0 violations.
- [ ] Manual test
- [ ] No need to test or manual test.
- Behavior changed:
- [ ] No.
- [x] Yes. Invalid Paimon system table reader options are visible in the
SQL error message.
- Does this need documentation?
- [x] No.
- [ ] Yes.
### 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]
