Re: PATCH: optimized DROP of multiple tables within a transaction

On 4.1.2013 17:42, Robert Haas wrote:
> On Mon, Dec 31, 2012 at 11:51 AM, Tomas Vondra <tv@fuzzy.cz> wrote:
>> I thought I followed the conding style - which guidelines have I broken?
> 
> +	/* If there are no non-local relations, then we're done. Release the memory
> +	 * and return. */
> 
> Multi-line comments should start with a line containing only /* and
> end with a line containing only */.
> 
> +DropRelFileNodeAllBuffers(RelFileNodeBackend * rnodes, int nnodes)
> and
> +rnode_comparator(const void * p1, const void * p2)
> 
> The extra spaces after the asterisks should be removed.
> 
> +void smgrdounlinkall(SMgrRelation * rels, int nrels, bool isRedo)
> +{
> 
> void should be on a line by itself.
> 
> Sorry to nitpick.

No, thanks for the nitpicking! Code style is important.

> As for BSEARCH_LIMIT, I don't have a great idea - maybe just
> DROP_RELATIONS_BSEARCH_LIMIT?

Sounds good. I've changed the name and fixed the codestyle issues in the
attached version of the patch.

Tomas