On Wed, Apr 29, 2026 at 7:12 AM Ajin Cherian <itsajin@gmail.com> wrote:
>
> On Wed, Apr 29, 2026 at 12:17 AM Bharath Rupireddy
> <bharath.rupireddyforpostgres@gmail.com> wrote:
> >
> >
> >
> > Please find the attached v4 patch. Thank yoU!
> >
>
> One small comment. The includes need to be in alphabetical order.
> injection_point.h should come after fmgroids.h
>
> #include "utils/fmgroids.h"
> #include "utils/lsyscache.h"
> #include "utils/rel.h"
> +#include "utils/injection_point.h"
> #include "utils/syscache.h"
>
+1
Also there is a trailing whitespace issue while applying the patch.
Other than these, the patch looks good.
thanks
Shveta