+1 for this, and it should be simple to implement since the actual
compression is decided based on the msgr2 connection feature negotiation.
If we have an alternative we can just say that future OSDs will not accept
this compression during connection feature negotiation.
One more point - we should be very careful not to compress the data in the
buffer list too soon (and please excuse my ignorance about the exact data
layering implementation in Ceph) - some feature have to look into the data
(dedup...), so we need to make sure that we don't compress above the level
that these features need the data.
Regards,
Josh
On Thu, Feb 11, 2021 at 2:13 AM Sage Weil <sweil(a)redhat.com> wrote:
Another thought: if ensure that this is gated by a
feature flag (or
similar) that is easily removed later (e.g., a msgr2 feature instead of a
ceph feature bit), we could go with this solution now, and if/when we have
something we like better gracefully remove this support in a future release.
_______________________________________________
Dev mailing list -- dev(a)ceph.io
To unsubscribe send an email to dev-leave(a)ceph.io