On Fri, Sep 20, 2019 at 12:38 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> > One small thing:
> >
> > add_int_reloption(bl_relopt_kind, buf,
> > "Number of bits generated for each index column",
> > - DEFAULT_BLOOM_BITS, 1, MAX_BLOOM_BITS);
> > + DEFAULT_BLOOM_BITS, 1, MAX_BLOOM_BITS,
> > + AccessExclusiveLock);
> > Do we need a comment to explain why we're using AccessExclusiveLock in
> > this case?
>
> Because that's the safest default to use here? That seemed obvious to
> me.
Okay. Sounds good.
--
Thanks & Regards,
Kuntal Ghosh
EnterpriseDB: http://www.enterprisedb.com