Re: [HACKERS] Make pg_basebackup -x stream the default

Usability review

============

Patch sounds like a good idea and does what it supposed to do. /me in DBA hat will be happy to have it.

However, it makes '-x' parameter a bit confusing/surprising: specifying it will be equivalent to '-X fetch' which is surprising regression from the new default.

One comment about docs:

         Includes the required transaction log files (WAL files) in the

         backup. This will include all transaction logs generated during

-        the backup. If this option is specified, it is possible to start

-        a postmaster directly in the extracted directory without the need

-        to consult the log archive, thus making this a completely standalone

-        backup.

+        the backup. Unless the option <literal>none</literal> is specified,

+        it is possible to start a postmaster directly in the extracted

+        directory without the need to consult the log archive, thus

+        making this a completely standalone backup.

        </para>

        <para>

I suggest "method <literal>none</literal>" instead of "option <literal>none</literal>". I found the word "option" confusing in that sentence.