Re: Running the fdw test from the terminal crashes into the core-dump

Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> After having pushed that, I wonder if we should document this. It seems
> quite the minor thing, but I'm sure somebody will complain if we don't.

Yup, no doubt.

> I propose the attached. (Extra context so that the full paragraph can
> be read from the comfort of your email program.)

This reads awkwardly to me. I think it'd be better to construct it
so that DO NOTHING's requirement is stated exactly parallel to the other
three clause types, more or less as attached.

> (While at it, I found the placement of the previous-to-last sentence in
> that paragraph rather strange, so I moved it to the end.)

Agreed, and done in my version too.

BTW, if you read it without paying attention to markup, you'll notice
that we are saying things like

If you specify an insert action, you must have the INSERT
privilege on the target_table_name.

which is fairly nonsensical: we don't define privileges on the name
of something. While I've not done anything about that here,
I wonder if we shouldn't just write "privilege on the target table"
without any special markup.

regards, tom lane