Thanks On Tue, Mar 11, 2025 at 7:51 PM Junwang Zhao <zhjw...@gmail.com> wrote:
> > Here is a new version with Amit's fix and my trivial refactors. > > 0001-0010 is the same as Ashutosh's last email > 0011 is Amit's fix of the crash in ExecCheckPermissions I think that fix is correct and it fixes the crash. I need to think more about it - especially whether there are more places where we are missing RTE_GRAPH_TABLE. But will do that as time permits. > 0012 is a trivial fix that remove the test with default collation, or > it will fail CI, see[1] > 0013-0015 are some trivial fix and refactor, feel free to incorporate > or drop when you review them. > > [1]: > https://api.cirrus-ci.com/v1/artifact/task/5290818690351104/testrun/build/testrun/regress/regress/regression.diffs > Thanks Junwang. I have added your patches to my local repository. Next time I post an updated set, I will post it along with your patches. Will merge them into the original patches as time permits. -- Best Wishes, Ashutosh Bapat
