Skip to content

refactor(reads): remove leader consistency selector (EN-1946) - #1893

Open
gfyrag wants to merge 1 commit into
feat/en-1946-inspect-index-alignmentfrom
feat/en-1946-remove-leader-consistency
Open

refactor(reads): remove leader consistency selector (EN-1946)#1893
gfyrag wants to merge 1 commit into
feat/en-1946-inspect-index-alignmentfrom
feat/en-1946-remove-leader-consistency

Conversation

@gfyrag

@gfyrag gfyrag commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Stack 5/7 for EN-1946. Removes the unreleased leader consistency selector while retaining stale. Unknown leader metadata now follows the existing default linearizable route.

@NumaryBot

NumaryBot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

✅ Approve — automated review

The leader selector is consistently removed, while stale and default linearizable routing remain intact. No actionable correctness issues were found.

No findings.

@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.20%. Comparing base (94e9664) to head (e538a48).

Additional details and impacted files
@@                           Coverage Diff                            @@
##           feat/en-1946-inspect-index-alignment    #1893      +/-   ##
========================================================================
+ Coverage                                 77.04%   77.20%   +0.16%     
========================================================================
  Files                                       478      478              
  Lines                                     51513    51502      -11     
========================================================================
+ Hits                                      39689    39764      +75     
+ Misses                                     8366     8275      -91     
- Partials                                   3458     3463       +5     
Flag Coverage Δ
e2e 77.20% <100.00%> (+0.16%) ⬆️
scenario 77.20% <100.00%> (+0.16%) ⬆️
unit 77.20% <100.00%> (+0.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gfyrag
gfyrag force-pushed the feat/en-1946-remove-leader-consistency branch from 13a8aab to e538a48 Compare September 4, 2026 11:25
@gfyrag
gfyrag force-pushed the feat/en-1946-inspect-index-alignment branch from 3ddec50 to 94e9664 Compare September 4, 2026 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants