Thread

1. BUG #5315: Unlisted keyword WINDOW

   sulfinu@gmail.com — 2010-02-05T11:45:30Z

   The following bug has been logged online:
   
   Bug reference:      5315
   Logged by:          
   Email address:      sulfinu@gmail.com
   PostgreSQL version: 8.4.2
   Operating system:   Gentoo Linux
   Description:        Unlisted keyword WINDOW
   Details: 
   
   According to the official documentation
   (http://www.postgresql.org/docs/8.4/interactive/sql-keywords-appendix.html),
   the SQL reserved keyword WINDOW is totally uninteresting to PostgreSQL.
   
   But in real life the parser has a different opinion. Just try this:
   create table a (
   window integer,
   c varchar(20));
   
   So, either correct the parser (is there any use for the word WINDOW in
   Postgres?) or the documentation.
   
   

2. Re: BUG #5315: Unlisted keyword WINDOW

   Bruce Momjian <bruce@momjian.us> — 2010-02-05T19:40:52Z

   sulfinu@gmail.com wrote:
   > 
   > The following bug has been logged online:
   > 
   > Bug reference:      5315
   > Logged by:          
   > Email address:      sulfinu@gmail.com
   > PostgreSQL version: 8.4.2
   > Operating system:   Gentoo Linux
   > Description:        Unlisted keyword WINDOW
   > Details: 
   > 
   > According to the official documentation
   > (http://www.postgresql.org/docs/8.4/interactive/sql-keywords-appendix.html),
   > the SQL reserved keyword WINDOW is totally uninteresting to PostgreSQL.
   > 
   > But in real life the parser has a different opinion. Just try this:
   > create table a (
   > window integer,
   > c varchar(20));
   > 
   > So, either correct the parser (is there any use for the word WINDOW in
   > Postgres?) or the documentation.
   
   Ah, quite interesting.  I grabed all the keywords in our SGML docs
   listed as somehow reserved in Postgres, and then ran "SELECT word FROM 
   pg_get_keywords() WHERE catcode != 'U'" to get all the keywords from our
   C code, and then compared the two.  The two missing keywords were OVER,
   and WINDOW, as you mentioned.
   
   I have applied the attached patch to CVS HEAD and 8.4.X to properly
   document our keywords.
   
   -- 
     Bruce Momjian  <bruce@momjian.us>        http://momjian.us
     EnterpriseDB                             http://enterprisedb.com
   
     + If your life is a hard drive, Christ can be your backup. +