Message-ID: <gh-pgjdbc-pgjdbc-2918-c1634053539@github.com>
From: "davecramer (@davecramer)" <noreply+davecramer@github.com>
To: "pgjdbc/pgjdbc" <noreply+pgjdbc-pgjdbc@github.com>
Date: Thu, 13 Jul 2023 11:09:52 +0000
Subject: Re: [pgjdbc/pgjdbc] issue #2918: Set defaultRowFetchSize=10000 by default to improve app stability
In-Reply-To: <gh-pgjdbc-pgjdbc-2918@github.com>
References: <gh-pgjdbc-pgjdbc-2918@github.com>
List-Id: <gh-pgjdbc-pgjdbc.github.com>
X-GitHub-Author-Login: davecramer
X-GitHub-Comment-Id: 1634053539
X-GitHub-Comment-Type: issue_comment
X-GitHub-Issue: 2918
X-GitHub-Repo: pgjdbc/pgjdbc
X-GitHub-Type: comment
X-GitHub-Url: https://github.com/pgjdbc/pgjdbc/issues/2918#issuecomment-1634053539
Content-Type: text/plain; charset=utf-8

> I think `-1` for `executeLargeUpdate` and `executeUpdate` is a bug. For DMLs we should have server-side information on "the number of modified rows", and we should just return it in `executeUpdate`.

Well we don't get server side information if fetchsize is set to a non zero positive number and we are returning a result set. It has been said that if you are returning a resultset then executeUpdate is the wrong function to call. Either way there's little the driver can do.