Re: DatabaseMetaData.getIndexInfo() added - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: DatabaseMetaData.getIndexInfo() added
Date
Msg-id 200105152346.f4FNkif13888@candle.pha.pa.us
Whole thread Raw
In response to DatabaseMetaData.getIndexInfo() added  (William Webber <william@ferengi.live.com.au>)
List pgsql-patches
I am about to apply this to 7.2.

> Hi all!
>
> Attached is a patch that implements DatabaseMetaData.getIndexInfo()
> for the JDBC2 driver.  It also fixes a compile error under jikes by
> casting characters in case statements of a byte-compared switch to
> bytes.
>
> A few notes on getIndexInfo():
>
> 1.) getArray() is not implemented for the postgresql JDBC driver yet,
> so getIndexInfo() parses the pg_index.indkey field into separate
> integers by hand.

OK.

> 2.) I have guessed that if pg_index.indisclustered is "false", then
> the index is "hashed"; if not, line 2561 of the resultant class
> should have "tableIndexOther" rather than "tableIndexHashed".

I will fix this. I know how to do it because I just did this for ODBC.

> 3.) I didn't know what sort sequence (if any) was used in indexes, so
> I have set it to "null" (unknown) on line 2566.

OK.

> 4.) For "CARDINALITY" (number of unique index items, the 11th field of
> the ResultSet returned by getIndexInfo()) I have used
> pg_classes.reltuples.

Is that perhaps the number of columns in the index?

>
> I have tested this method, but hardly extensively.  Is there a proper
> regression test suite for the JDBC driver that tests can be added to?

Does this get added to JDBC1 too?

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

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Re: GIST question
Next
From: Hiroshi Inoue
Date:
Subject: Re: [ODBC] Fix for SQLStatistics