Can someone tackles this and supply a patch?


> Would some JDBC hacker develop a patch for the following issue?  The
> change is just barely large enough that I don't want to commit untested
> code for it --- but not having a Java development environment at hand,
> I can't test the updated code.
> 
> The problem is in DatabaseMetaData.java (same code in both jdbc1 and
> jdbc2, looks like).  It does direct access to pg_description that isn't
> right anymore.  In getTables, instead of
> 
>       java.sql.ResultSet dr = connection.ExecSQL("select description from 
>pg_description where objoid="+r.getInt(2));
> 
> it should be
> 
>       java.sql.ResultSet dr = connection.ExecSQL("select 
>obj_description("+r.getInt(2)+",'pg_class')");
> 
> In getColumns, the change is a little more involved, because
> pg_attribute doesn't have an OID column anymore.  The initial query
> can't fetch a.oid, but should fetch a.attrelid instead, and then the
> pg_description query should become
> 
>       java.sql.ResultSet dr = connection.ExecSQL("select 
>col_description("+r.getInt(1)+","+r.getInt(5)+")");
> 
> (col_description takes the table OID and the column's attnum).
> 
> The reason this is more than a 3-line change is that it should be done
> either the old way or the new way depending on whether server version >=
> 7.2 or not, for backwards-compatibility of the driver.
> 
> It's possible there are other similar changes needed that I missed in a
> quick lookover.
> 
> So, would some enterprising person fix the JDBC code to work with CVS
> tip, and submit a patch?
> 
>               thanks, tom lane
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Don't 'kill -9' the postmaster
> 

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  [EMAIL PROTECTED]               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
    (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

Reply via email to