Thanks for the feedback. dj08: I wonder whether we should use 1s for the interval combined with > offloading enabled by default. The offloading preserves the coordinator > threads while providing back-pressure because a group can have only one > assignment inflight. Going from 5s to 1s would ensure that groups remain > more dynamic.
I've updated the default to 1s. dj09: Should we have min/max configs on the server for the new interval, > similar to other intervals? I've added those too. I've renamed min.assignment.interval.ms to assignment.interval.ms, otherwise the names look weird. dj10: What's the reason for using -1 by default for the Timestamp field? It seems we can just use zero. I wanted a value to indicate "no timestamp". 0 works just as well as -1. I've updated the KIP. > dj11: I wonder whether we should have a plan to sunset > the *.assignor.offload.enable=false too. I believe that this will hurt > users more than anything else. We could perhaps mention this in a future > work section. Updated the Compatibility, Deprecation, and Migration Plan to mention this. dj12: In the migration section, we should also mention the offloading enabled by default. Updated the Compatibility, Deprecation, and Migration Plan section Thanks, Sean On Thu, Feb 12, 2026 at 1:08 PM David Jacot via dev <[email protected]> wrote: > Hi Sean, > > Thanks for the update. I have a few more comments to consider: > > dj08: I wonder whether we should use 1s for the interval combined with > offloading enabled by default. The offloading preserves the coordinator > threads while providing back-pressure because a group can have only one > assignment inflight. Going from 5s to 1s would ensure that groups remain > more dynamic. > dj09: Should we have min/max configs on the server for the new interval, > similar to other intervals? > dj10: What's the reason for using -1 by default for the Timestamp field? It > seems we can just use zero. > dj11: I wonder whether we should have a plan to sunset > the *.assignor.offload.enable=false too. I believe that this will hurt > users more than anything else. We could perhaps mention this in a future > work section. > dj12: In the migration section, we should also mention the offloading > enabled by default. > > Best, > David > > On Tue, Feb 10, 2026 at 10:14 PM Sean Quah via dev <[email protected]> > wrote: > > > Thanks all for the feedback so far. I will start a vote within the next > few > > days unless anyone raises further feedback. > > > > Thanks, > > Sean > > > > On Mon, Jan 26, 2026 at 10:42 PM Dongnuo Lyu via dev < > [email protected] > > > > > wrote: > > > > > Thanks Sean for the update! > > > > > >
