pgsql/src/interfaces/jdbc/org/postgresql Conne ...

From: "Marc G(dot) Fournier" <scrappy(at)hub(dot)org>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql/src/interfaces/jdbc/org/postgresql Conne ...
Date: 2001-09-06 03:13:34
Message-ID: 200109060313.f863DY646801@hub.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

CVSROOT: /home/projects/pgsql/cvsroot
Module name: pgsql
Changes by: scrappy(at)hub(dot)org 01/09/05 23:13:34

Modified files:
src/interfaces/jdbc/org/postgresql: Connection.java
src/interfaces/jdbc/org/postgresql/jdbc1: Connection.java
src/interfaces/jdbc/org/postgresql/jdbc2: Connection.java

Log message:
Attached is my attempt to clean up the horrors of the ExecSQL() method in
the JDBC driver.

I've done this by extracting it into a new method object called
QueryExecutor (should go into org/postgresql/core/) and then taking it
apart into different methods in that class.

A short summary:

* Extracted ExecSQL() from Connection into a method object called
QueryExecutor.

* Moved ReceiveFields() from Connection to QueryExecutor.

* Extracted parts of the original ExecSQL() method body into smaller
methods on QueryExecutor.

* Bug fix: The instance variable "pid" in Connection was used in two
places with different meaning. Both were probably in dead code, but it's
fixed anyway.

Anders Bengtsson

Browse pgsql-committers by date

  From Date Subject
Next Message Marc G. Fournier 2001-09-06 03:15:43 pgsql/src/makefiles Makefile.unixware
Previous Message Marc G. Fournier 2001-09-06 03:11:59 pgsql/src/interfaces/jdbc/org/postgresql jdbc2 ...