So I’m just taking a quick peek at SerializationHeader and we already have a method for reading and writing a deletion time with offsets from EncodingStats.
So perhaps we simply have a bug where we are using DeletionTime Serializer instead of SerializationHeader.writeLocalDeletionTime? It looks to me like this is already available at most (perhaps all) of the relevant call sites. > On 29 Jun 2023, at 15:53, Josh McKenzie <jmcken...@apache.org> wrote: > > >> >> I would prefer we not plan on two distinct changes to this > I agree with this sentiment, and > >> +1, if you have time for this approach and no other in this window. > People are going to use 5.0 for awhile. Better to have an improvement in > their hands for that duration than no improvement at all IMO. Justifies the > cost of the double implementation and transitions to me. > >> On Tue, Jun 27, 2023, at 5:43 AM, Mick Semb Wever wrote: >> Just for completeness the change is a handful loc. The rest is added tests >> and we'd loose the sstable format change opportunity window. >> >> >> >> +1, if you have time for this approach and no other in this window. >> >> (If you have time for the other, or someone else does, then the technically >> superior approach should win) >> >> >