On Wed, Mar 22, 2023 2:53 PM Önder Kalacı <onderkal...@gmail.com> wrote: > > We don't really need to, if you check the first patch, we don't have DROP for > generated case. I mostly > wanted to make the test a little more interesting, but it also seems to be a > little confusing. > > Now attaching v2 where we do not drop the columns. I don't have strong > preference on > which patch to proceed with, mostly wanted to attach this version to progress > faster (in case > you/Amit considers this one better). >
Thanks for updating the patches. The v2 patch LGTM. Regards, Shi Yu