Gabriel39 opened a new pull request, #69551:
URL: https://github.com/apache/doris/pull/69551
### What problem does this PR solve?
Iceberg has broad regression coverage, but feature names alone miss state
transitions and weak oracles. For example, expiring fresh snapshots with a
cutoff in 2020 never exercises ref retention.
Add 45 connector unit tests, two external regression suites, and an
inventory mapping the published 84 operations plus read, version, model,
lineage, type, scanner and storage dimensions to existing tests and remaining
gaps. The tests cover snapshot/schema/ref isolation, atomic writes and failure
paths, nested type metadata, exact predicate semantics, partition
serialization, actual ref retention, and file positions across deletes and
history.
Nine individually disabled reproductions preserve correct assertions for
five confirmed issue families: DECIMAL-to-DOUBLE pruning, lineage-name
collisions during a V2-to-V3 upgrade, ignored per-operation isolation
properties, mixed MERGE affected-row counts, and expiration counters built from
stale metadata. These were run and observed failing before disabling.
Production behavior is unchanged.
### Release note
None
### Validation
- Full connector reactor: `mvn -f fe/pom.xml -pl :fe-connector-iceberg -am
test -DfailIfNoTests=false -Dmaven.build.cache.enabled=false` with the matching
Thrift 0.24.0 compiler: BUILD SUCCESS, including FE Checkstyle.
- Iceberg connector: 1,734 discovered, 1,720 passed, 14 skipped. New tests:
45 discovered, 36 passed, 9 intentionally disabled defect reproductions.
- Both Groovy suites compile with Groovy 4.0.19. They use existing
REST/MinIO/Spark fixtures and dynamic comparisons, with no handwritten golden
results.
- SQL suites were not executed: the managed QA/Jenkins endpoints are
unreachable from the validation host. BE unit tests and the full
backend/deployment matrix were not run.
### Check List (For Author)
- Test
- [x] Regression test (added; compilation checked; SQL execution pending)
- [x] Unit Test
- Behavior changed:
- [x] No.
- Does this need documentation?
- [x] No. Test coverage inventory is included.
--
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]
