Skip to content

WPB-12109: Migrate Elasticsearch client to upstream bloodhound v1.0.0.0 - #5561

Open
blackheaven wants to merge 1 commit into
developfrom
gdifolco/WPB-12109-bloodhound-upstream
Open

blackheaven wants to merge 1 commit into
developfrom
gdifolco/WPB-12109-bloodhound-upstream

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-12109

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@blackheaven
blackheaven requested review from a team as code owners September 24, 2026 15:58
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Sep 24, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-12109-bloodhound-upstream branch from 2f38b8b to 4cb3475 Compare September 24, 2026 21:58
@blackheaven blackheaven changed the title WPB-12109: Migrate Elasticsearch client to upstream bloodhound v1.0.0.0 WPB-12109: Migrate Elasticsearch client to upstream bloodhound v1.0.0.0 and OpenSearch 1.3 Sep 25, 2026
Replace our wireapp/bloodhound fork pin with upstream bitemyapp/bloodhound
v1.0.0.0 (rev-pinned in flake.nix) and port all call sites to its API:

- Drop the typed-API surface (MappingName/searchByType/putNamedMapping) in
  favor of untyped /{index}/_doc and PUT /{index}/_mapping endpoints;
  requires ES >= 7 (or >= 6.7 with untyped indices).
- Handle the new error model: runBH returns Either EsError, client functions
  throw EsError; normalize via tryEsError in the polysemy interpreters.
  Version-conflict (409) tolerance is preserved via error status checks.
- Version user documents with external_gte instead of strictly-greater
  external: re-indexing with an equal version now overwrites (behavior
  change, covered by a new mock-interpreter unit test).
- Adapt to upstream changes: abstract IndexName (mkIndexName/unIndexName),
  Key-typed term fields, Maybe BooleanOperator, 3-field IndexSettings,
  EsResult-based getDocument, new MonadBH class (Backend/dispatch).
- Bump dev chart elasticsearch-ephemeral to ES 7.17.26 (chart 0.0.43).
@blackheaven
blackheaven force-pushed the gdifolco/WPB-12109-bloodhound-upstream branch 2 times, most recently from afd7a8c to 993910f Compare October 20, 2026 10:53
@blackheaven blackheaven changed the title WPB-12109: Migrate Elasticsearch client to upstream bloodhound v1.0.0.0 and OpenSearch 1.3 WPB-12109: Migrate Elasticsearch client to upstream bloodhound v1.0.0.0 Oct 20, 2026
@blackheaven
blackheaven added this pull request to stack #5613 October 20, 2026 13:07

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants