Skip to content

Combine hash computation with bloom filter writes/reads#140969

Merged
fcofdez merged 4 commits intoelastic:mainfrom
fcofdez:inline-hash
Jan 21, 2026
Merged

Combine hash computation with bloom filter writes/reads#140969
fcofdez merged 4 commits intoelastic:mainfrom
fcofdez:inline-hash

Conversation

@fcofdez
Copy link
Copy Markdown
Contributor

@fcofdez fcofdez commented Jan 20, 2026

This commit combines the hash computation with bloom filter writes and reads.
The aim of this is to help the JIT compiler to inline the code from the previous two loops.

This commit combines the hash computation with bloom filter writes and
reads. The aim of this is to help the JIT compiler to inline the
code from both loops.
@fcofdez fcofdez requested review from burqen and tlrx January 20, 2026 14:18
@fcofdez fcofdez added >enhancement >non-issue :Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. Team:Distributed Indexing (obsolete) Meta label for Distributed Indexing team. Obsolete. Please do not use. labels Jan 20, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing)

1 similar comment
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing)

Copy link
Copy Markdown
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

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

LGTM

@fcofdez
Copy link
Copy Markdown
Contributor Author

fcofdez commented Jan 21, 2026

@elasticmachine update branch

@fcofdez
Copy link
Copy Markdown
Contributor Author

fcofdez commented Jan 21, 2026

@elasticmachine update branch

@elasticmachine
Copy link
Copy Markdown
Collaborator

merge conflict between base and head

@fcofdez
Copy link
Copy Markdown
Contributor Author

fcofdez commented Jan 21, 2026

@elasticmachine update branch

@fcofdez fcofdez merged commit 53972ef into elastic:main Jan 21, 2026
35 checks passed
szybia added a commit to szybia/elasticsearch that referenced this pull request Jan 21, 2026
…-tests

* upstream/main: (104 commits)
  Partition time-series source (elastic#140475)
  Mute org.elasticsearch.xpack.esql.heap_attack.HeapAttackSubqueryIT testManyRandomKeywordFieldsInSubqueryIntermediateResultsWithSortManyFields elastic#141083
  Reindex relocation: skip nodes marked for shutdown (elastic#141044)
  Make fails on fixture caching not fail image building (elastic#140959)
  Add multi-project tests for get and list reindex (elastic#140980)
  Painless docs overhaul (reference) (elastic#137211)
  Panama vector implementation of codePointCount (elastic#140693)
  Enable PromQL in release builds (elastic#140808)
  Update rest-api-spec for Jina embedding task (elastic#140696)
  [CI] ShardSearchPhaseAPMMetricsTests testUniformCanMatchMetricAttributesWhenPlentyOfDocumentsInIndex failed (elastic#140848)
  Combine hash computation with bloom filter writes/reads (elastic#140969)
  Refactor posting iterators to provide more information (elastic#141058)
  Wait for cluster to recover to yellow before checking index health (elastic#141057) (elastic#141065)
  Fix repo analysis read count assertions (elastic#140994)
  Fixed a bug in logsdb rolling upgrade sereverless tests involving par… (elastic#141022)
  Fix readiness edge case on startup (elastic#140791)
  PromQL: fix quantile function (elastic#141033)
  ignore `mmr` command for check (in development) (elastic#140981)
  Use Double.compare to compare doubles in tdigest.Sort (elastic#141049)
  Migrate third party module tests using legacy test clusters framework (elastic#140991)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >enhancement >non-issue Team:Distributed Indexing (obsolete) Meta label for Distributed Indexing team. Obsolete. Please do not use. v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants