> Patch V2 addressed the comments. Overall this LGTM.
I still see a "no real storage" in v2 that should be removed from the documentation. + Foreign tables have no real storage in PostgreSQL. + Inapplicable options: <literal>INCLUDING INDEXES</literal>, <literal>INCLUDING STORAGE</literal>, I think the test coverage to check for the negative conditions only is enough. Regards, Sami
