Patch for JDBC driver not returning correct # of records deleted

Started by Barry Lindalmost 26 years ago2 messagespatches
Jump to latest
#1Barry Lind
barry@xythos.com

I had a problem with the JDBC driver (in 7.0.2) always returning 1 as
the row count for deletes (whether the actual delete affected 0 or more
rows). In looking through the source this appears to be a simple one
line code fix.

Change line 370 in src/interfaces/jdbc/org/postgresql/Connection.java
v1.1 from:
if(recv_status.startsWith("INSERT") ||
recv_status.startsWith("UPDATE")) {
to:
if(recv_status.startsWith("INSERT") ||
recv_status.startsWith("UPDATE") || recv_status.startsWith("DELETE")) {

thanks,
--Barry

#2Bruce Momjian
bruce@momjian.us
In reply to: Barry Lind (#1)
Re: Patch for JDBC driver not returning correct # of records deleted

I had a problem with the JDBC driver (in 7.0.2) always returning 1 as
the row count for deletes (whether the actual delete affected 0 or more
rows). In looking through the source this appears to be a simple one
line code fix.

Change line 370 in src/interfaces/jdbc/org/postgresql/Connection.java
v1.1 from:
if(recv_status.startsWith("INSERT") ||
recv_status.startsWith("UPDATE")) {
to:
if(recv_status.startsWith("INSERT") ||
recv_status.startsWith("UPDATE") || recv_status.startsWith("DELETE")) {

thanks,
--Barry

If I look at the code in the current CVS tree, I see a test only for
INSERT, and not INSERT and UPDATE. Can you supply a context diff that I
can apply? That would help me be sure I am changing the right lines.
Thanks.

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