I think it makes sense to do this in the same PR because those jump tables will no longer be needed since clients with the corresponding versions will no longer be allowed. Similarly there may be Command classes that are no longer needed because they're only used in old jump tables.
On 12/9/20, 9:36 AM, "Jacob Barrett" <jabarr...@vmware.com> wrote: > On Dec 9, 2020, at 8:09 AM, Bruce Schuchardt <bru...@vmware.com> wrote: > > We should also get rid of the old jump tables in CommandInitializer Would it make sense to do that in the same PR that tackles setting the min client version in the handshake or should that be a separate task? -Jake