On Tue, Sep 9, 2025 at 12:00 PM Fujii Masao <[email protected]> wrote:
>
> On Thu, Sep 4, 2025 at 6:00 PM jian he <[email protected]> wrote:
> > in _tocEntryRestorePass
> > if we do
> >
> > if ((strcmp(te->desc, "COMMENT") == 0 ||
> > strcmp(te->desc, "SECURITY LABEL") == 0) &&
> > strncmp(te->tag, "EVENT TRIGGER ", 14) == 0)
> > return RESTORE_PASS_POST_ACL;
> >
> > then RestorePass related comments also need to be adjusted for security
> > label?
>
> Could you clarify which comments should be updated for SECURITY LABEL?
> I don't see any references to COMMENT in RestorePass, so it doesn't look
> like any updates are needed there for SECURITY LABEL either. But maybe
> I'm missing something...
>
typedef enum
{
RESTORE_PASS_MAIN = 0, /* Main pass (most TOC item types) */
RESTORE_PASS_ACL, /* ACL item types */
RESTORE_PASS_POST_ACL, /* Event trigger and matview refresh items */
#define RESTORE_PASS_LAST RESTORE_PASS_POST_ACL
} RestorePass;
I thought we needed to change the comments:
" /* Event trigger and matview refresh items */".
looking at the code again, we don't need to do that.
Overall, all the patches look good to me.
(I didn't test back branch related tests though)
