fix: [StorageV2] Access future result to get exception if any (#43613)

Related to #43584

When `LoadWithStrategy` throw exception, the ex was wrapped in the
returned future. If the future is not handled, this exception would be
ignored.

This patch add `future.get()` to get exception if any.

Signed-off-by: Congqi Xia <congqi.xia@zilliz.com>
This commit is contained in:
congqixia 2025-07-28 22:33:35 +08:00 committed by GitHub
parent bd31b32167
commit f666d89919
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -231,6 +231,10 @@ GroupChunkTranslator::get_cells(const std::vector<cachinglayer::cid_t>& cids) {
filled_cids.insert(cid);
}
}
// access underlying feature to get exception if any
load_future.get();
// Verify all requested cids have been filled
for (auto cid : cids) {
AssertInfo(filled_cids.find(cid) != filled_cids.end(),