Skip to content

fix: auto-index silently never fires on batched increments - #7

Open
gamandeepsingh wants to merge 1 commit into
solana-rpc:mainfrom
gamandeepsingh:fix/auto-index-threshold-skip
Open

fix: auto-index silently never fires on batched increments#7
gamandeepsingh wants to merge 1 commit into
solana-rpc:mainfrom
gamandeepsingh:fix/auto-index-threshold-skip

Conversation

@gamandeepsingh

Copy link
Copy Markdown

*count == threshold only matched when the count landed exactly on the threshold, which worked for one-by-one increments but silently dropped entries when a batch flush pushed the count past it in one go (e.g. 0→15 with threshold=10).

Changed to *count < threshold + increment — equivalent to checking previous_count < threshold — so the entry is added to the priority queue on the genuine first crossing, regardless of batch size, without re-triggering after the queue consumes it.

Closes: #2

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.

crates/query-tracker/src/tracker.rs:145 - auto-index silently never fires

1 participant