Message-ID: <gh-pgjdbc-pgjdbc-3509-rc1942108706@github.com>
From: "davecramer (@davecramer)" <noreply+davecramer@github.com>
To: "pgjdbc/pgjdbc" <noreply+pgjdbc-pgjdbc@github.com>
Date: Wed, 05 Feb 2025 01:01:09 +0000
Subject: Re: [pgjdbc/pgjdbc] PR #3509: Add back application name setting
In-Reply-To: <gh-pgjdbc-pgjdbc-3509-rc1942012936@github.com>
References: <gh-pgjdbc-pgjdbc-3509@github.com> <gh-pgjdbc-pgjdbc-3509-rc1942012936@github.com>
List-Id: <gh-pgjdbc-pgjdbc.github.com>
X-GitHub-Author-Login: davecramer
X-GitHub-Comment-Id: 1942108706
X-GitHub-Comment-Type: review_comment
X-GitHub-Commit: 9ef77ad39125a283fe56d42899eac94cea0f2afa
X-GitHub-Issue: 3509
X-GitHub-Path: pgjdbc/src/main/java/org/postgresql/core/v3/ConnectionFactoryImpl.java
X-GitHub-Repo: pgjdbc/pgjdbc
X-GitHub-Type: review_comment
X-GitHub-Url: https://github.com/pgjdbc/pgjdbc/pull/3509#discussion_r1942108706
Content-Type: text/plain; charset=utf-8

(on pgjdbc/src/main/java/org/postgresql/core/v3/ConnectionFactoryImpl.java)

> I don't know your connection string, for me it works unless I set the assumeMinServerVersion connection property to 9.0 or greater, if I omit that property the name gets set, just too late for the initial connection logging.
> 
> It does appear that you could still skip the initial queries if the assume version was >= 12. Everything else would already be set correctly.

Why do you need this at initial connection logging ?