This is a discussion on Re: [COMMITTERS] pgsql: RESET SESSION, plus related new DDL commands. within the pgsql Hackers forums, part of the PostgreSQL category; --> Neil Conway wrote: > Log Message: > ----------- > RESET SESSION, plus related new DDL commands. This phrase is ...
| |||||||
| FAQ | Members List | Calendar | Search | Today's Posts | Mark Forums Read |
| ||||
| Neil Conway wrote: > Log Message: > ----------- > RESET SESSION, plus related new DDL commands. This phrase is missing a verb: The default value is defined as the value that the parameter would have had, if no <command>SET</> ever been issued for it in the current session. /pgsql/doc/src/sgml/ref/reset.sgml I find this markup strange: <synopsis> SET SESSION AUTHORIZATION DEFAULT; RESET ALL; DEALLOCATE ALL; CLOSE ALL; UNLISTEN *; RESET PLANS; RESET TEMP; </synopsis> shouldn't it be using <programlisting> ? In ResetTempTableNamespace(void), shouldn't it be using myTempNamespace instead of the SysCache lookup? -- Alvaro Herrera http://www.CommandPrompt.com/ The PostgreSQL Company - Command Prompt, Inc. ---------------------------(end of broadcast)--------------------------- TIP 3: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq |
| ||||
| On Thu, 2007-04-12 at 11:45 -0400, Alvaro Herrera wrote: > This phrase is missing a verb: > [...] > I find this markup strange: > [...] > In ResetTempTableNamespace(void), shouldn't it be using myTempNamespace > instead of the SysCache lookup? All fair points: I've applied the attached patch. Thanks for the review. -Neil ---------------------------(end of broadcast)--------------------------- TIP 3: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq |