On Mon, Dec 6, 2021 at 6:18 PM Euler Taveira <eu...@eulerto.com> wrote: > > On Mon, Dec 6, 2021, at 3:44 AM, Amit Kapila wrote: > > True, but that is the main reason the review and development are being > done as separate sub-features. I suggest still keeping the similar > separation till some of the reviews of each of the patches are done, > otherwise, we need to rethink how to divide for easier review. We need > to retain the 0005 patch because that handles many problems without > which the main patch is incomplete and buggy w.r.t replica identity. > > IMO we should merge sub-features as soon as we reach consensus. Every new > sub-feature breaks comments, tests and documentation if you want to remove or > rearrange patches. >
I agree that there is some effort but OTOH, it gives the flexibility to do a focussed review and as soon as some patch is ready or close to ready we can merge in the main patch. This was just a humble suggestion based on how this patch was making progress and how it has helped to keep some parts separate by allowing different people to work on different parts of the problem. > It seems I misread 0005. I agree that it is important. I'll > check it. > Okay, thanks! -- With Regards, Amit Kapila.