+1 On Fri, Jan 20, 2017 at 4:51 PM, Ismael Juma <ism...@juma.me.uk> wrote:
> Good catch, Colin. +1 to editing the wiki to match the desired behaviour > and what was implemented in 0.10.1. > > Ismael > > On Sat, Jan 21, 2017 at 12:19 AM, Colin McCabe <cmcc...@apache.org> wrote: > > > Hi all, > > > > While looking at some code related to KIP-74, I noticed a slight > > discrepancy between the text on the wiki and the implementation. The > > wiki says that "If max_bytes is Int.MAX_INT, new request behaves exactly > > like old one." This would mean that if there was a single message that > > was larger than the maximum bytes per partition, zero messages would be > > returned, and clients would throw MessageSizeTooLargeException. > > However, the code does not implement this. Instead, it implements the > > "new" behavior where the client always gets at least one message. > > > > The new behavior seems to be more desirable, since clients do not "get > > stuck" on messages that are too big. I propose that we edit the wiki to > > reflect the implemented behavior by deleting the references to special > > behavior when max_bytes is MAX_INT. > > > > cheers, > > Colin > > >
