Gabriel39 opened a new pull request, #69601:
URL: https://github.com/apache/doris/pull/69601
### What problem does this PR solve?
Related PR: #69312
Port #69312 from `branch-4.2` to current `master`.
- Remove Hive ACID read transactions, shared read locks, write-ID snapshots,
delta-file listing and the BE transactional reader. Reject full ACID and
insert-only reads in the Hive connector.
- Adapt the changes to master's connector modules and unified reader
initialization interfaces. Preserve ordinary Hive scanning and Iceberg/Paimon
ORC position-delete and deletion-vector handling.
- Reject standalone Avro and deprecated table markers before file access or
metadata-only counts. Keep LakeSoul unsupported and remove its obsolete test
service and examples.
- Preserve Iceberg/Hudi Avro metadata dependencies. The Thrift changes add
deprecation comments only; every definition and numeric ID is unchanged.
### Release note
Remove support for Hive transactional reads, including insert-only tables,
and standalone Avro file reads. LakeSoul remains unsupported. Iceberg and Hudi
Avro metadata parsing remains supported.
### Validation
- Hive connector: all 490 tests passed; HMS client: all 155 tests passed.
- 20 targeted FE core tests passed: deprecated-read rejection,
partition-path parsing, generic file scan planning and Hive streaming scans.
- FE Checkstyle passed with zero violations for every affected FE module.
- All 12 affected C++ source/header files passed clang-format 16; build
hygiene and shell syntax checks passed.
- Syntax-only compilation passed for all 6 affected BE production
translation units and all 4 affected BE test translation units. Isolated checks
relaxed pre-existing unused-constant/shadow warnings. Full BE linking and
runtime tests remain for CI.
- Verified that Thrift changes are comments only and checked the affected
definitions against branch-3.1/4.0/4.1/4.2. `git diff --check` passed.
### Check List (For Author)
- Test
- [x] Unit Test
- [x] Manual test: formatting, syntax, build hygiene and Thrift
compatibility checks described above.
- Behavior changed:
- [x] Yes. Reject unsupported external reads explicitly.
- Does this need documentation?
- [x] No. This ports the existing removal to master.
--
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]
