Message-ID: From: "reda-alaoui (@reda-alaoui)" To: "pgjdbc/pgjdbc" Date: Mon, 16 Mar 2026 09:10:28 +0000 Subject: Re: [pgjdbc/pgjdbc] PR #3964: Allow setting of KeepAlive options in JDBC driver In-Reply-To: References: List-Id: X-GitHub-Author-Login: reda-alaoui X-GitHub-Comment-Id: 2939038566 X-GitHub-Comment-Type: review_comment X-GitHub-Commit: 113b0e34eefce6a93fae808d61b06a97ad0c6272 X-GitHub-Issue: 3964 X-GitHub-Line: 89 X-GitHub-Path: pgjdbc/src/main/java11/org/postgresql/core/ExtendedSocketOptionAccessorImpl.java X-GitHub-Repo: pgjdbc/pgjdbc X-GitHub-Type: review_comment X-GitHub-Url: https://github.com/pgjdbc/pgjdbc/pull/3964#discussion_r2939038566 Content-Type: text/plain; charset=utf-8 (on pgjdbc/src/main/java11/org/postgresql/core/ExtendedSocketOptionAccessorImpl.java:89) But this causes an issue. How to copy socket options in `PGStream` ( https://github.com/pgjdbc/pgjdbc/pull/3964/changes#diff-1c9bbe3aa0c1f2d0d7a6ecef10d981c2be064a526e8f91e2b47b04823fd50dedR160-R162 ) without failure? At this step, we don't know if `ExtendedSocketOptions` have been explicitly required by the library consumer.