Hi Yugo,

Your patch change looks good.

Yes Amit, I agree that SPI_execute_snapshot() comments do provide some context on AFTER triggers, but I still feel the newly added comment in ri_PerformCheck() gives additional context on why the fire_triggers is set to false.

Regards,
Surya Poondla