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

Reply via email to