branch-4.1 [fix](variant) Bind Variant search to nested indexes by eldenmoon · Pull Request #63765 · apache/doris

eldenmoon · 2026-05-27T14:01:57Z

cherry-pick #63660 to branch-4.1

### What problem does this PR solve? Issue Number: N/A Related PR: apache#63660 Problem Summary: Backport apache#63660 to branch-4.1. Bind Variant inverted-index search to the resolved scalar or nested Variant index reader, map nested leaf results back to the expected document scope, and preserve null bitmap semantics for empty bitset truth bitmaps. Adapt the segment index iterator call to the branch-4.1 ColumnReader API. Cherry-picked from commits 8310d28 and 315ad31. ### Release note Fix Variant inverted-index search binding for scalar and nested Variant paths. ### Check List (For Author) - Test: - Unit Test: ./run-be-ut.sh --run --filter='*Variant*:FunctionSearchTest.TestBuildLeafQueryDirectUnknownClauseUsesLeafMapper:FunctionSearchNestedTest.*:BitSetQueryTest.EmptyTruthBitmapPreservesNullBitmap' - Behavior changed: Yes. Fixes Variant inverted-index search binding and null bitmap handling. - Does this need documentation: No

hello-stephen · 2026-05-27T14:02:03Z

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

eldenmoon · 2026-05-28T01:22:28Z

run buildall

Copilot

Pull request overview

This PR cherry-picks Variant inverted-index search fixes into branch-4.1, separating Variant binding/nested search logic from the generic search() function and adding diagnostics for Variant index binding.

Changes:

Adds variant_inverted_index_search support for Variant subcolumn binding, direct BKD reads, UNKNOWN bitmap handling, and nested-doc mapping.
Updates search() query construction and BitSetQuery/BitSetWeight to preserve null bitmap semantics.
Adds diagnostics and focused unit tests for Variant binding, nested mapping, and empty-truth/null-bitmap behavior.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
`be/src/exprs/function/function_search.cpp`	Integrates Variant resolver/evaluator, UNKNOWN queries, and direct-reader leaf query handling.
`be/src/exprs/function/function_search.h`	Moves Variant-specific resolver/evaluator APIs into the new header.
`be/src/exprs/function/variant_inverted_index_search.cpp`	Implements Variant field binding, nested leaf mapping, and nested search evaluation.
`be/src/exprs/function/variant_inverted_index_search.h`	Declares Variant search binding and nested mapping APIs.
`be/src/exprs/vsearch.cpp`	Adds Variant binding diagnostics during input collection and search evaluation.
`be/src/storage/index/inverted/query_v2/bit_set_query/bit_set_query.h`	Adds null-bitmap storage to `BitSetQuery`.
`be/src/storage/index/inverted/query_v2/bit_set_query/bit_set_weight.h`	Preserves scorers when only null bitmap data is present.
`be/src/storage/index/inverted/inverted_index_stats.h`	Stores capped Variant binding diagnostics.
`be/src/storage/index/inverted/inverted_index_profile.h`	Publishes binding diagnostics to runtime profile info strings.
`be/src/storage/segment/segment.cpp`	Adds index-file probe and iterator creation diagnostics.
`be/src/storage/segment/segment_iterator.cpp`	Passes stats into Variant subcolumn index discovery and logs iterator diagnostics.
`be/src/storage/segment/variant/variant_column_reader.cpp`	Adds direct/inherited/missing subcolumn index candidate diagnostics.
`be/src/storage/segment/variant/variant_column_reader.h`	Extends subcolumn index lookup API with optional stats.
`be/test/exprs/function/function_search_test.cpp`	Adds tests for Variant missing fields, resolver selection, and direct scalar reads.
`be/test/exprs/function/function_search_nested_test.cpp`	Adds tests for nested doc mapping and evaluator behavior.
`be/test/storage/index/inverted/query_v2/boolean_query_test.cpp`	Adds coverage for empty truth bitmap with preserved null bitmap.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+ const bool is_text_field =
+ column_type != nullptr && is_string_type(column_type->get_storage_field_type());
+ auto fb_it = _field_binding_map.find(field_name);
+ std::string analyzer_key;
+ if (is_text_field && is_variant_sub && fb_it != _field_binding_map.end() &&
+ fb_it->second->__isset.index_properties && !fb_it->second->index_properties.empty()) {


@@ -252,6 +303,11 @@ Status VSearchExpr::evaluate_inverted_index(VExprContext* context, uint32_t segm
 if (bundle.iterators.empty() && !is_nested_query) {


+ FieldReaderBinding binding;
+ binding.logical_field_name = "var.items.active";
+ binding.stored_field_name = "1.var.items.active";
+ binding.stored_field_wstr = L"1.var.items.active";
+ binding.column_type = bool_type;
+ binding.query_type = InvertedIndexQueryType::MATCH_PHRASE_QUERY;
+ binding.state = SearchFieldBindingState::BOUND;
+ TabletIndex index_meta;
+ binding.inverted_reader = std::make_shared<DummyInvertedIndexReader>(&index_meta);
+
+ std::string key = resolver.binding_key_for("1.var.items.active",
+ InvertedIndexQueryType::MATCH_PHRASE_QUERY);
+ binding.binding_key = key;
+ resolver._cache[key] = binding;


+ FieldReaderBinding binding;
+ binding.logical_field_name = "var.items.active";
+ binding.stored_field_name = "1.var.items.active";
+ binding.stored_field_wstr = L"1.var.items.active";
+ binding.column_type = bool_type;
+ binding.query_type = InvertedIndexQueryType::MATCH_ANY_QUERY;
+ binding.state = SearchFieldBindingState::BOUND;
+ TabletIndex index_meta;
+ binding.inverted_reader = std::make_shared<DummyInvertedIndexReader>(&index_meta);
+
+ std::string key =
+ resolver.binding_key_for("1.var.items.active", InvertedIndexQueryType::MATCH_ANY_QUERY);
+ binding.binding_key = key;
+ resolver._cache[key] = binding;
+
+ inverted_index::query_v2::QueryPtr out;
+ std::string out_binding_key;
+ Status st = function_search->build_leaf_query(clause, context, resolver, &out, &out_binding_key,
+ "OR", 0, 10);
+ ASSERT_TRUE(st.ok());
+ ASSERT_NE(out, nullptr);
+ EXPECT_EQ(key, out_binding_key);


+ FieldReaderBinding binding;
+ binding.logical_field_name = "var.items.flags.level";
+ binding.stored_field_name = "1.var.items.flags.level";
+ binding.stored_field_wstr = L"1.var.items.flags.level";
+ binding.column_type = int_type;
+ binding.query_type = InvertedIndexQueryType::MATCH_ANY_QUERY;
+ binding.state = SearchFieldBindingState::BOUND;
+ TabletIndex index_meta;
+ binding.inverted_reader = std::make_shared<DummyInvertedIndexReader>(&index_meta);
+
+ std::string key = resolver.binding_key_for("1.var.items.flags.level",
+ InvertedIndexQueryType::MATCH_ANY_QUERY);
+ binding.binding_key = key;
+ resolver._cache[key] = binding;
+
+ inverted_index::query_v2::QueryPtr out;
+ std::string out_binding_key;
+ Status st = function_search->build_leaf_query(clause, context, resolver, &out, &out_binding_key,
+ "OR", 0, 10);
+ ASSERT_TRUE(st.ok());
+ ASSERT_NE(out, nullptr);
+ EXPECT_EQ(key, out_binding_key);


yiguolei · 2026-05-28T06:02:03Z

skip buildall

github-actions · 2026-05-28T06:03:50Z

PR approved by at least one committer and no changes requested.

github-actions · 2026-05-28T06:03:54Z

PR approved by anyone and no changes requested.

eldenmoon marked this pull request as ready for review May 28, 2026 01:26

eldenmoon requested a review from yiguolei as a code owner May 28, 2026 01:26

CopilotAI review requested due to automatic review settings May 28, 2026 01:26

Copilotstarted reviewing on behalf of eldenmoon May 28, 2026 01:26 View session

eldenmoon changed the title ~~[fix](variant) Bind Variant search to nested indexes~~branch-4.1 [fix](variant) Bind Variant search to nested indexesMay 28, 2026

CopilotAI reviewed May 28, 2026
View reviewed changes

yiguolei approved these changes May 28, 2026
View reviewed changes

github-actionsBot added the approved Indicates a PR has been approved by one committer. label May 28, 2026

github-actionsBot added the reviewed label May 28, 2026

eldenmoon merged commit b360a98 into apache:branch-4.1May 28, 2026
37 of 40 checks passed

Navigation Menu

Uh oh!

branch-4.1 [fix](variant) Bind Variant search to nested indexes - #63765

branch-4.1 [fix](variant) Bind Variant search to nested indexes#63765
eldenmoon merged 1 commit into
apache:branch-4.1from
eldenmoon:branch-pick-63660-4.1

eldenmoon commented May 27, 2026•
edited
Loading

Uh oh!

hello-stephen commented May 27, 2026

Uh oh!

eldenmoon commented May 28, 2026

Uh oh!

CopilotAI left a comment

Uh oh!

yiguolei commented May 28, 2026

Uh oh!

github-actionsBot commented May 28, 2026

Uh oh!

github-actionsBot commented May 28, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		@@ -252,6 +303,11 @@ Status VSearchExpr::evaluate_inverted_index(VExprContext* context, uint32_t segm
		if (bundle.iterators.empty() && !is_nested_query) {

Uh oh!

Conversation

eldenmoon commented May 27, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hello-stephen commented May 27, 2026

Uh oh!

eldenmoon commented May 28, 2026

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

yiguolei commented May 28, 2026

Uh oh!

github-actionsBot commented May 28, 2026

Uh oh!

github-actionsBot commented May 28, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

eldenmoon commented May 27, 2026•
edited
Loading