On Wed, Mar 22, 2023 2:53 PM Önder Kalacı <onderkalaci(at)gmail(dot)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