Patrick Steinhardt <ps@xxxxxx> writes: > Hi, > > this is the second version of my patch series that aims to improve the > way reftable stack perform compaction. > > Changes compared to v1: > > - Extract a test helper function that sets up a stack with N tables > containing refs. > > - Reuse file descriptor that we have already stored in a local > variable instead of calling `lock_file_fd()` a second time. > > - Remove a no-op change in the last patch. > > - Add a comment explaining why we have to allocate N+1 many table > names. > > - Some typo fixes. > > Thanks! I haven't reviewed v1, but did skim through it. I left some comments, (maybe they were already asked/answered) overall the series looks great to me. [snip]
Attachment:
signature.asc
Description: PGP signature
