Re: 2018-03 Commitfest starts tomorrow

David Steele <david(at)pgmasters(dot)net> writes:
> OK, here's my break down after a brief review of all the patches in the
> list. Many of the patches were trivial/small, bug fixes, had history,
> or related to builds, tests, or docs.

> There are still 13 patches that are medium/large (in terms of impact,
> not size) from my analysis, and new.

FWIW, several of these are part of Peter's continuing work on procedures,
and thus arguably not totally new:

> chained transactions
> INOUT parameters in procedures
> SET TRANSACTION in PL/pgSQL
> PL/pgSQL nested CALL with transactions
> committing inside cursor loop

So I'd be inclined to cut these a little slack in hopes of having a more
complete/coherent procedure feature in v11. Still, they are showing up
late in the dev cycle, so they shouldn't get a whole lot of slack.

> Changing the autovacuum launcher scheduling; oldest table first algorithm

Definitely too late for this cycle.

> Let's remove DSM_INPL_NONE?

This discussion has been ongoing for awhile, and it does have some of the
nature of a bug fix since if we don't do this we'll instead have to deal
with bugs in the places that fail to cope.

> [PROPOSAL] Nepali Snowball dictionary?
> Opclass parameters (recommend RWF or push by me)
> Parallel Dump to /dev/null (recommend push by Andres)

Push or RWF all of these, IMO.

> The following patches seem OK to me, but I'm happy for feedback.

I'm okay with leaving these alive for now. Some of 'em perhaps should
fail on merit, but that is a case-by-case issue rather than something
to handle through this discussion.

regards, tom lane