Re: Patch for JDBC to update some comments

Lists: pgsql-patches
From: Barry Lind <barry(at)xythos(dot)com>
To: pgsql-patches(at)postgresql(dot)org
Subject: Patch for JDBC to update some comments
Date: 2001-08-16 05:46:07
Message-ID: 3B7B5E1F.2040808@xythos.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

This patch updates some comments in the DatabaseMetaData classes to
reflect a mail thread that discussed our conformance (or lack thereof)
to the SQL92 spec.

thanks,
--Barry

Attachment Content-Type Size
patch.diff text/plain 4.5 KB

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Barry Lind <barry(at)xythos(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Patch for JDBC to update some comments
Date: 2001-08-16 13:29:39
Message-ID: 200108161329.f7GDTdZ11979@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches


Your patch has been added to the PostgreSQL unapplied patches list at:

http://candle.pha.pa.us/cgi-bin/pgpatches

I will try to apply it within the next 48 hours.

> This patch updates some comments in the DatabaseMetaData classes to
> reflect a mail thread that discussed our conformance (or lack thereof)
> to the SQL92 spec.
>
> thanks,
> --Barry

> *** ./interfaces/jdbc/org/postgresql/jdbc1/DatabaseMetaData.java.orig Sat Aug 4 12:32:04 2001
> --- ./interfaces/jdbc/org/postgresql/jdbc1/DatabaseMetaData.java Wed Aug 15 22:39:55 2001
> ***************
> *** 680,686 ****
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 680,691 ----
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true. We currently
> ! * report false until 'schema' support is added. Then this
> ! * should be changed to return true, since we will be mostly
> ! * compliant (probably more compliant than many other databases)
> ! * And since this is a requirement for all JDBC drivers we
> ! * need to get to the point where we can return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 726,732 ****
> }
>
> /**
> ! * Is some form of outer join supported? From my knowledge, nope.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 731,737 ----
> }
>
> /**
> ! * Is some form of outer join supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 737,744 ****
> }
>
> /**
> ! * Are full nexted outer joins supported? Well, we dont support any
> ! * form of outer join, so this is no as well
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 742,748 ----
> }
>
> /**
> ! * Are full nexted outer joins supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 749,756 ****
> }
>
> /**
> ! * Is there limited support for outer joins? (This will be true if
> ! * supportFullOuterJoins is true)
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 753,759 ----
> }
>
> /**
> ! * Is there limited support for outer joins?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> *** ./interfaces/jdbc/org/postgresql/jdbc2/DatabaseMetaData.java.orig Wed Aug 15 22:25:44 2001
> --- ./interfaces/jdbc/org/postgresql/jdbc2/DatabaseMetaData.java Wed Aug 15 22:37:10 2001
> ***************
> *** 680,686 ****
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 680,691 ----
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true. We currently
> ! * report false until 'schema' support is added. Then this
> ! * should be changed to return true, since we will be mostly
> ! * compliant (probably more compliant than many other databases)
> ! * And since this is a requirement for all JDBC drivers we
> ! * need to get to the point where we can return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 726,732 ****
> }
>
> /**
> ! * Is some form of outer join supported? From my knowledge, nope.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 731,737 ----
> }
>
> /**
> ! * Is some form of outer join supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 737,744 ****
> }
>
> /**
> ! * Are full nexted outer joins supported? Well, we dont support any
> ! * form of outer join, so this is no as well
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 742,748 ----
> }
>
> /**
> ! * Are full nexted outer joins supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 749,756 ****
> }
>
> /**
> ! * Is there limited support for outer joins? (This will be true if
> ! * supportFullOuterJoins is true)
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 753,759 ----
> }
>
> /**
> ! * Is there limited support for outer joins?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs

>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: Have you searched our list archives?
>
> http://www.postgresql.org/search.mpl

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


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Barry Lind <barry(at)xythos(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Patch for JDBC to update some comments
Date: 2001-08-17 14:46:21
Message-ID: 200108171446.f7HEkLQ21874@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches


Thanks. Patch applied.

> This patch updates some comments in the DatabaseMetaData classes to
> reflect a mail thread that discussed our conformance (or lack thereof)
> to the SQL92 spec.
>
> thanks,
> --Barry

> *** ./interfaces/jdbc/org/postgresql/jdbc1/DatabaseMetaData.java.orig Sat Aug 4 12:32:04 2001
> --- ./interfaces/jdbc/org/postgresql/jdbc1/DatabaseMetaData.java Wed Aug 15 22:39:55 2001
> ***************
> *** 680,686 ****
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 680,691 ----
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true. We currently
> ! * report false until 'schema' support is added. Then this
> ! * should be changed to return true, since we will be mostly
> ! * compliant (probably more compliant than many other databases)
> ! * And since this is a requirement for all JDBC drivers we
> ! * need to get to the point where we can return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 726,732 ****
> }
>
> /**
> ! * Is some form of outer join supported? From my knowledge, nope.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 731,737 ----
> }
>
> /**
> ! * Is some form of outer join supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 737,744 ****
> }
>
> /**
> ! * Are full nexted outer joins supported? Well, we dont support any
> ! * form of outer join, so this is no as well
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 742,748 ----
> }
>
> /**
> ! * Are full nexted outer joins supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 749,756 ****
> }
>
> /**
> ! * Is there limited support for outer joins? (This will be true if
> ! * supportFullOuterJoins is true)
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 753,759 ----
> }
>
> /**
> ! * Is there limited support for outer joins?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> *** ./interfaces/jdbc/org/postgresql/jdbc2/DatabaseMetaData.java.orig Wed Aug 15 22:25:44 2001
> --- ./interfaces/jdbc/org/postgresql/jdbc2/DatabaseMetaData.java Wed Aug 15 22:37:10 2001
> ***************
> *** 680,686 ****
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 680,691 ----
>
> /**
> * Does this driver support the ANSI-92 entry level SQL grammar?
> ! * All JDBC Compliant drivers must return true. We currently
> ! * report false until 'schema' support is added. Then this
> ! * should be changed to return true, since we will be mostly
> ! * compliant (probably more compliant than many other databases)
> ! * And since this is a requirement for all JDBC drivers we
> ! * need to get to the point where we can return true.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 726,732 ****
> }
>
> /**
> ! * Is some form of outer join supported? From my knowledge, nope.
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 731,737 ----
> }
>
> /**
> ! * Is some form of outer join supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 737,744 ****
> }
>
> /**
> ! * Are full nexted outer joins supported? Well, we dont support any
> ! * form of outer join, so this is no as well
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 742,748 ----
> }
>
> /**
> ! * Are full nexted outer joins supported?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> ***************
> *** 749,756 ****
> }
>
> /**
> ! * Is there limited support for outer joins? (This will be true if
> ! * supportFullOuterJoins is true)
> *
> * @return true if so
> * @exception SQLException if a database access error occurs
> --- 753,759 ----
> }
>
> /**
> ! * Is there limited support for outer joins?
> *
> * @return true if so
> * @exception SQLException if a database access error occurs

>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: Have you searched our list archives?
>
> http://www.postgresql.org/search.mpl

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