Recent patch to DatabaseMetaData

Started by Peter T Mountover 25 years ago10 messagescomitters
Jump to latest
#1Peter T Mount
peter@retep.org.uk

I missed the commit message about 3 days ago to the file
src/interfaces/jdbc/org/postgresql/jdbc2/DatabaseMetaData.java

Can it be backed out please and the patch sent to me? I've got a commit that's
failing and there's a lot of methods in that class that have now changed (ie:
JDBC will now report that outer joins are now possible) and there's the
TestCase for that class that can't go in because it reports the wrong
information.

Thanks, Peter

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/

#2Bruce Momjian
bruce@momjian.us
In reply to: Peter T Mount (#1)
Re: Recent patch to DatabaseMetaData

[ Charset ISO-8859-1 unsupported, converting... ]

I missed the commit message about 3 days ago to the file
src/interfaces/jdbc/org/postgresql/jdbc2/DatabaseMetaData.java

Can it be backed out please and the patch sent to me? I've got a commit that's
failing and there's a lot of methods in that class that have now changed (ie:
JDBC will now report that outer joins are now possible) and there's the
TestCase for that class that can't go in because it reports the wrong
information.

Here is the patch. I applied it to DatabaseMetaData.java in postgresql/
and jdbc1 and jdbc2. It was a cleanup of the code to no longer handle
large object files, because they are all stored in a single table now.

Hard to imagine that caused the problems you reported. I can back it
out, but it hits a couple of files. Please let me know. Recent logs
for that file are:

revision 1.12
date: 2001/02/09 22:23:47; author: momjian; state: Exp; lines: +4 -6
Remove last code that assumed xinv/xinx are large object files.
----------------------------
revision 1.11
date: 2001/01/24 09:22:01; author: peter; state: Exp; lines: +251
-251
Removed the 8k row limit reported by DatabaseMetaData
----------------------------
revision 1.10
date: 2000/11/25 04:32:12; author: momjian; state: Exp; lines: +8 -8
Allow jdbc to return proper server version number

Anders Bengtsson
----------------------------

-- 
  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

Attachments:

/bjm/difftext/plainDownload+10-10
#3Peter T Mount
peter@retep.org.uk
In reply to: Bruce Momjian (#2)
Re: Recent patch to DatabaseMetaData

Quoting Bruce Momjian <pgman@candle.pha.pa.us>:

Hard to imagine that caused the problems you reported. I can back it
out, but it hits a couple of files. Please let me know. Recent logs
for that file are:

The file that's important is the jdbc/DatabaseMetaData.java file as that's the
one I've been updating recently. I've got to syncronise the jdbc1 files when I
get JDK1.1 back online.

I've already made the modifications to my copy here on that file. Can you
backout just the changes to that one file?

Peter

revision 1.12
date: 2001/02/09 22:23:47; author: momjian; state: Exp; lines: +4
-6
Remove last code that assumed xinv/xinx are large object files.
----------------------------
revision 1.11
date: 2001/01/24 09:22:01; author: peter; state: Exp; lines: +251
-251
Removed the 8k row limit reported by DatabaseMetaData
----------------------------
revision 1.10
date: 2000/11/25 04:32:12; author: momjian; state: Exp; lines: +8
-8
Allow jdbc to return proper server version number

Anders Bengtsson
----------------------------

-- 
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

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/

#4Bruce Momjian
bruce@momjian.us
In reply to: Peter T Mount (#3)
Re: Recent patch to DatabaseMetaData

[ Charset ISO-8859-1 unsupported, converting... ]

Quoting Bruce Momjian <pgman@candle.pha.pa.us>:

Hard to imagine that caused the problems you reported. I can back it
out, but it hits a couple of files. Please let me know. Recent logs
for that file are:

The file that's important is the jdbc/DatabaseMetaData.java file as that's the
one I've been updating recently. I've got to syncronise the jdbc1 files when I
get JDK1.1 back online.

I've already made the modifications to my copy here on that file. Can you
backout just the changes to that one file?

Sure, which file do you want backed out?

./org/postgresql/jdbc1/DatabaseMetaData.java
./org/postgresql/jdbc2/DatabaseMetaData.java
./postgresql/jdbc1/DatabaseMetaData.java
./postgresql/jdbc2/DatabaseMetaData.java

-- 
  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
#5Peter T Mount
peter@retep.org.uk
In reply to: Bruce Momjian (#4)
Re: Recent patch to DatabaseMetaData

Quoting Bruce Momjian <pgman@candle.pha.pa.us>:

This one please:

./org/postgresql/jdbc2/DatabaseMetaData.java

Thanks.

./postgresql/jdbc1/DatabaseMetaData.java
./postgresql/jdbc2/DatabaseMetaData.java

Hmmm, I thought I deleted those from CVS (it's old obsolete pre 7.0 code).

Peter

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/

#6Bruce Momjian
bruce@momjian.us
In reply to: Peter T Mount (#5)
Re: Recent patch to DatabaseMetaData

OK, backed out and patch attached.

[ Charset ISO-8859-1 unsupported, converting... ]

Quoting Bruce Momjian <pgman@candle.pha.pa.us>:

This one please:

./org/postgresql/jdbc2/DatabaseMetaData.java

Thanks.

./postgresql/jdbc1/DatabaseMetaData.java
./postgresql/jdbc2/DatabaseMetaData.java

Hmmm, I thought I deleted those from CVS (it's old obsolete pre 7.0 code).

Peter

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/

-- 
  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

Attachments:

/bjm/difftext/plainDownload+10-10
#7Bruce Momjian
bruce@momjian.us
In reply to: Peter T Mount (#5)
Re: Recent patch to DatabaseMetaData

Yes, I asked recently if these should be removed. You want me to remove
them?

./postgresql/jdbc1/DatabaseMetaData.java
./postgresql/jdbc2/DatabaseMetaData.java

Hmmm, I thought I deleted those from CVS (it's old obsolete pre 7.0 code).

Peter

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/

-- 
  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
#8Peter T Mount
peter@retep.org.uk
In reply to: Bruce Momjian (#7)
Re: Recent patch to DatabaseMetaData

Quoting Bruce Momjian <pgman@candle.pha.pa.us>:

Yes, I asked recently if these should be removed. You want me to
remove
them?

Yes please. the entire ./postgresql/ directory is dead. I left it there for the
7.0 series but for 7.1 it's obsolete.

Thanks, Peter

./postgresql/jdbc1/DatabaseMetaData.java
./postgresql/jdbc2/DatabaseMetaData.java

Hmmm, I thought I deleted those from CVS (it's old obsolete pre 7.0

code).

Peter

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/

-- 
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

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/

#9Bruce Momjian
bruce@momjian.us
In reply to: Peter T Mount (#5)
Re: Recent patch to DatabaseMetaData

./postgresql/jdbc1/DatabaseMetaData.java
./postgresql/jdbc2/DatabaseMetaData.java

Hmmm, I thought I deleted those from CVS (it's old obsolete pre 7.0 code).

Removed.

-- 
  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
#10Peter T Mount
peter@retep.org.uk
In reply to: Bruce Momjian (#9)
Re: Recent patch to DatabaseMetaData

Quoting Bruce Momjian <pgman@candle.pha.pa.us>:

./postgresql/jdbc1/DatabaseMetaData.java
./postgresql/jdbc2/DatabaseMetaData.java

Hmmm, I thought I deleted those from CVS (it's old obsolete pre 7.0

code).

Removed.

Thanks. My stuff now committed.

Peter

--
Peter Mount peter@retep.org.uk
PostgreSQL JDBC Driver: http://www.retep.org.uk/postgres/
RetepPDF PDF library for Java: http://www.retep.org.uk/pdf/