yezhizi opened a new pull request, #3184:
URL: https://github.com/apache/kvrocks/pull/3184

   An edge case existed (due to the handling of [`latest_bucket_idx` in the 
downstream 
meta](https://github.com/apache/kvrocks/blob/4025a6d23698b6bd5383511f259d10988fe7dc0e/src/types/redis_timeseries.h#L91-L98))
 where inserting data into an empty series with the first `bucket index > 0` 
would incorrectly aggregate an empty or invalid bucket to the downstream 
series. The insertion logic has been corrected to fix this bug.
   
   - Add a `DownstreamUpsertArgs` struct to contain a flag that indicates this 
edge case.
   - Correctly update `latest_bucket_idx` in the downstream meta.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to