Skip to content

obf-bitset-index#25250

Closed
Gateway07 wants to merge 14 commits into
masterfrom
obf-bitset-index
Closed

obf-bitset-index#25250
Gateway07 wants to merge 14 commits into
masterfrom
obf-bitset-index

Conversation

@Gateway07

Copy link
Copy Markdown
Contributor
  • add OBF bitset index impl
  • add OBF bitset index fixes
  • suffixesBitsetIndex, extraSuffix, poiIndInBlock fixes
  • Cross-category suffix policy fix

#25065 (comment)

@Gateway07
Gateway07 requested a review from vshcherb June 11, 2026 10:43
if (matcher.matches(name.toLowerCase(Locale.ROOT))) {
return true;
}
List<String> queryTokens = splitAndNormalize(query);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

too much extra code it's not needed for now. and probably incorrect

CollatorStringMatcher matcher = new CollatorStringMatcher(query, req.matcherMode);
long indexOffset = codedIS.getTotalBytesRead();
TIntLongHashMap offsetsMap = new TIntLongHashMap();
Map<Integer, Set<Integer>> poiIndexesByOffset = new HashMap<>();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To review use TIntLongHashMap, or TLongHashSet

@Gateway07 Gateway07 closed this Jul 1, 2026
@Gateway07
Gateway07 deleted the obf-bitset-index branch July 16, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants