Re: Add option --drop-cascade for pg_dump/restore

> 2) I felt pg_dump will include the cascade option for plain format and
> pg_restore will include the cascade option from pg_restore for other
> formats. If my understanding is correct, should we document this?

I may not understand it correctly, are you saying
pg_dump will include the cascade option only for plain format, or
pg_dump will enable the cascade option for plain by default?

> 4) Is it possible to add a few tests for this?

Looks like tests should be added to
`src/bin/pg_dump/t/002_pg_dump.pl`, I'll try to add some.

vignesh C <vignesh21@gmail.com> 于2021年7月13日周二 下午10:23写道：
>
> On Fri, Jul 2, 2021 at 12:11 PM Haotian Wu <whtsky@gmail.com> wrote:
> >
> > Hi,
> >
> > I agree that —drop-cascade does not make sense for pg_dumpall, so I removed them.
> >
> > > are we expecting more things to appear after the semi-colon?
> >
> > No, I was just trying to “reuse” original statement as much as possible. Append “\n” manually should also do the
job,and I’ve updated the patch as you suggests. 
>
> 1) This change is not required as it is not supported for pg_dumpall
> +++ b/doc/src/sgml/ref/pg_dumpall.sgml
> @@ -289,6 +289,16 @@ PostgreSQL documentation
>        </listitem>
>       </varlistentry>
>
> +     <varlistentry>
> +      <term><option>--drop-cascade</option></term>
> +      <listitem>
> +       <para>
> +        Use <literal>CASCADE</literal> to drop database objects.
> +        This option is not valid unless <option>--clean</option> is
> also specified.
> +       </para>
> +      </listitem>
> +     </varlistentry>
> +
>
> 2) I felt pg_dump will include the cascade option for plain format and
> pg_restore will include the cascade option from pg_restore for other
> formats. If my understanding is correct, should we document this?
>
> 3) This change is not required
>
> destroyPQExpBuffer(ftStmt);
>                                                         pg_free(dropStmtOrig);
>                                                 }
> +
>                                         }
>
> 4) Is it possible to add a few tests for this?
>
> Regards,
> Vignesh