mirror of
https://github.com/milvus-io/milvus.git
synced 2026-07-21 02:05:41 +00:00
fix: [cp3.0] avoid structured binding capture in OpenMP lambdas (#51549)
Cherry-pick from master pr: #51302 issue: #51231 Use init-captures for structured-binding field IDs in OpenMP lambdas. The 3.0 adaptation also fixes the matching JSON stats callback introduced by the 3.0 runtime-state backport. ## Test Plan - `/opt/homebrew/opt/llvm@18/bin/clang-format -i --style=file internal/core/src/segcore/ChunkedSegmentSealedImpl.cpp` - `git diff --check` - `ninja -C cmake_build -j1 -v src/segcore/CMakeFiles/milvus_segcore.dir/ChunkedSegmentSealedImpl.cpp.o` Signed-off-by: sijie-ni-0214 <sijie.ni@zilliz.com>
This commit is contained in:
@@ -5524,7 +5524,7 @@ ChunkedSegmentSealedImpl::LoadBatchJsonKeyIndexes(
|
||||
continue;
|
||||
}
|
||||
committer.Commit(
|
||||
[field_id, index = std::move(index)](
|
||||
[field_id = field_id, index = std::move(index)](
|
||||
RuntimeResourceState& runtime, PublishedSegmentState&) mutable {
|
||||
runtime.json_stats[field_id] = std::move(index);
|
||||
});
|
||||
@@ -7145,7 +7145,7 @@ ChunkedSegmentSealedImpl::FinalizeLoadDiffForReopen(
|
||||
CheckCancellation(op_ctx, id_, "ChunkedSegmentSealedImpl::ApplyLoadDiff()");
|
||||
for (const auto& [field_id, nested_paths] : diff.json_indexes_to_drop) {
|
||||
for (const auto& nested_path : nested_paths) {
|
||||
committer.Commit([&, field_id, nested_path](
|
||||
committer.Commit([&, field_id = field_id, nested_path](
|
||||
RuntimeResourceState& runtime,
|
||||
PublishedSegmentState& staged_state) {
|
||||
for (auto& retired :
|
||||
@@ -8249,7 +8249,7 @@ ChunkedSegmentSealedImpl::LoadBatchTextIndexes(
|
||||
field_id.get());
|
||||
auto future = pool.Submit([this,
|
||||
op_ctx,
|
||||
field_id,
|
||||
field_id = field_id,
|
||||
info = std::move(load_text_index_info),
|
||||
&segment_load_info,
|
||||
&committer]() mutable -> void {
|
||||
|
||||
Reference in New Issue
Block a user