liaoxin01 opened a new pull request, #68310:
URL: https://github.com/apache/doris/pull/68310
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary:
The synchronous CachedRemoteFileReader path constructed CacheContext without
copying the reader tablet ID. Cache misses were downloaded correctly, but the
created FileBlock and persisted block metadata used the default tablet ID 0.
Consequently, per-tablet cache inspection and TTL reconciliation could not find
those blocks.
This change propagates _tablet_id into the synchronous path cache context.
It also adds a unit test that forces the warm-up/synchronous write path and
verifies that the resulting block retains tablet ID 10086.
### Release note
Fix file cache blocks created by synchronous reads to retain their tablet ID.
### Check List (For Author)
- Test
- [ ] Regression test
- [x] Unit Test
- Added
AsyncCachedRemoteFileReaderTest.sync_write_path_preserves_tablet_id.
- The modified IO library and the new test object compile successfully.
- Full local test execution was blocked before linking by stale local
third-party Thrift/AWS SDK headers unrelated to this change.
- [ ] Manual test
- [ ] No need to test or manual test. Explain why:
- Behavior changed:
- [ ] No.
- [x] Yes. Synchronously populated cache blocks now retain their tablet
ownership metadata.
- Does this need documentation?
- [x] No.
- [ ] Yes.
--
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]
