On Tue, 2012-10-02 at 10:46 -0500, Karl O. Pinc wrote:
> I am now submitting patches to the commitfest
> for review.  (I'm not sure how I missed this.)

I prefer this version of the patch. I also attached an alternative
version that may address Tom's concern by noting that the OIDs are
hidden in the description.

Marking "ready for committer".

Regards,
        Jeff Davis
*** a/doc/src/sgml/catalogs.sgml
--- b/doc/src/sgml/catalogs.sgml
***************
*** 427,432 ****
--- 427,439 ----
      <tbody>
  
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>amname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 683,688 ****
--- 690,702 ----
      <tbody>
  
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>amopfamily</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-opfamily"><structname>pg_opfamily</structname></link>.oid</literal></entry>
***************
*** 819,824 ****
--- 833,845 ----
      <tbody>
  
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>amprocfamily</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-opfamily"><structname>pg_opfamily</structname></link>.oid</literal></entry>
***************
*** 902,907 ****
--- 923,935 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>adrelid</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-class"><structname>pg_class</structname></link>.oid</literal></entry>
***************
*** 1257,1262 ****
--- 1285,1298 ----
      </thead>
  
      <tbody>
+ 
+      <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
       <row>
        <entry><structfield>rolname</structfield></entry>
        <entry><type>name</type></entry>
***************
*** 1462,1467 ****
--- 1498,1510 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>castsource</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-type"><structname>pg_type</structname></link>.oid</literal></entry>
***************
*** 1577,1582 ****
--- 1620,1632 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>relname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 1984,1989 ****
--- 2034,2046 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>conname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 2250,2255 ****
--- 2307,2319 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>collname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 2350,2355 ****
--- 2414,2426 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>conname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 2443,2448 ****
--- 2514,2526 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>datname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 2652,2657 ****
--- 2730,2742 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>defaclrole</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-authid"><structname>pg_authid</structname></link>.oid</literal></entry>
***************
*** 3006,3011 ****
--- 3091,3103 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>enumtypid</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-type"><structname>pg_type</structname></link>.oid</literal></entry>
***************
*** 3079,3084 ****
--- 3171,3183 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>extname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 3175,3180 ****
--- 3274,3286 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>fdwname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 3267,3272 ****
--- 3373,3385 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>srvname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 3686,3691 ****
--- 3799,3811 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>lanname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 3886,3891 ****
--- 4006,4018 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>lomowner</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-authid"><structname>pg_authid</structname></link>.oid</literal></entry>
***************
*** 3938,3943 ****
--- 4065,4077 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>nspname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 4006,4011 ****
--- 4140,4152 ----
      <tbody>
  
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>opcmethod</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-am"><structname>pg_am</structname></link>.oid</literal></entry>
***************
*** 4104,4109 ****
--- 4245,4257 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>oprname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 4254,4259 ****
--- 4402,4414 ----
      <tbody>
  
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>opfmethod</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-am"><structname>pg_am</structname></link>.oid</literal></entry>
***************
*** 4438,4443 ****
--- 4593,4605 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>proname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 4830,4835 ****
--- 4992,5004 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>rulename</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 5499,5504 ****
--- 5668,5680 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>spcname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 5567,5572 ****
--- 5743,5755 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>tgrelid</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-class"><structname>pg_class</structname></link>.oid</literal></entry>
***************
*** 5752,5757 ****
--- 5935,5947 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>cfgname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 5891,5896 ****
--- 6081,6093 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>dictname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 5967,5972 ****
--- 6164,6176 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>prsname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 6057,6062 ****
--- 6261,6273 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>tmplname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 6124,6129 ****
--- 6335,6347 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>typname</structfield></entry>
        <entry><type>name</type></entry>
        <entry></entry>
***************
*** 6629,6634 ****
--- 6847,6859 ----
  
      <tbody>
       <row>
+       <entry><structfield>oid</structfield></entry>
+       <entry><type>oid</type></entry>
+       <entry></entry>
+       <entry>Row identifier (hidden attribute; must be explicitly selected)</entry>
+      </row>
+ 
+      <row>
        <entry><structfield>umuser</structfield></entry>
        <entry><type>oid</type></entry>
        <entry><literal><link linkend="catalog-pg-authid"><structname>pg_authid</structname></link>.oid</literal></entry>
-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to