On Fri, Sep 20, 2019 at 12:38 PM Michael Paquier <mich...@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
