Skip to content

Fix flaky SVS tiered flow test by removing incorrect flat buffer assertion#934

Open
meiravgri wants to merge 1 commit intomainfrom
meiravg_fix_flaky_svs_flw_test
Open

Fix flaky SVS tiered flow test by removing incorrect flat buffer assertion#934
meiravgri wants to merge 1 commit intomainfrom
meiravg_fix_flaky_svs_flw_test

Conversation

@meiravgri
Copy link
Copy Markdown
Collaborator

@meiravgri meiravgri commented Apr 14, 2026

Remove an incorrect assertion in search_insert that checked get_curr_bf_size() == 0 immediately after wait_for_index(1).

The flat buffer is not guaranteed to be empty at that point: vectors added during the current update job — but not enough to trigger another one — may still reside in the buffer. This caused flaky test failures.

Mark if applicable

  • This PR introduces API changes
  • This PR introduces serialization changes

@jit-ci
Copy link
Copy Markdown

jit-ci bot commented Apr 14, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@meiravgri meiravgri changed the title remove assert MOD-14826 fix assert added in #931 Apr 14, 2026
@meiravgri meiravgri changed the title MOD-14826 fix assert added in #931 Fix flaky SVS tiered flow test by removing incorrect flat buffer assertion Apr 14, 2026
@meiravgri meiravgri requested a review from ofiryanai April 14, 2026 17:10
@meiravgri meiravgri enabled auto-merge April 14, 2026 17:10
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.99%. Comparing base (d7a5afb) to head (c61d002).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #934   +/-   ##
=======================================
  Coverage   96.99%   96.99%           
=======================================
  Files         129      129           
  Lines        7576     7576           
=======================================
  Hits         7348     7348           
  Misses        228      228           

☔ View full report in Codecov by Sentry.
📢 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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant