On Tue, Sep 20, 2022 at 6:36 PM Aleksander Alekseev <aleksan...@timescale.com> wrote: > > Hi Amit, > > > I think it is better to use Size. Even though, it may not fail now as > > the size of names for origin will always be much lesser but it is > > better if we are consistent. If we agree with this, then as a first > > patch, we can make it to use Size in existing places and then > > introduce this new function. > > OK, here is the updated patchset. > > * 0001 replaces int's with Size's in the existing code > * 0002 applies Peter's patch on top of 0001 >
LGTM. Thanks! ----- Kind Regards, Peter Smith. Fujitsu Australia