You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It could cause some messages contain more entries than max_batch_size. But I guess no overflow issue is introduced. I think the behavior needs to be changed.
Maybe we can just add utils::limit_size(after_batched_msg.entries, self.max_msg_size). But we still get some extra cost from the self.raft_log.entries().
raft-rs/src/raft.rs
Lines 565 to 566 in f7f7d01
try_batch
doesn't check the entries size after batched which might cause the overflow.Do we need to worry about this?
@hicqu @BusyJay
The text was updated successfully, but these errors were encountered: