pgjdbc/pgjdbc GitHub issues and pull requests (mirror)help / color / mirror / Atom feed
[pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl 6+ messages / 2 participants [nested] [flat]
* [pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl @ 2026-02-05 11:20 "davecramer (@davecramer)" <[email protected]> 0 siblings, 0 replies; 6+ messages in thread From: davecramer (@davecramer) @ 2026-02-05 11:20 UTC (permalink / raw) To: pgjdbc/pgjdbc <[email protected]> ^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: [pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl @ 2026-02-05 11:33 ` "vlsi (@vlsi)" <[email protected]> 4 siblings, 0 replies; 6+ messages in thread From: vlsi (@vlsi) @ 2026-02-05 11:33 UTC (permalink / raw) To: pgjdbc/pgjdbc <[email protected]> (on pgjdbc/src/main/java/org/postgresql/core/v3/ConnectionFactoryImpl.java:224) This looks wrong. It should better be `protocolMajor =...` `protocolMinor =...` ^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: [pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl @ 2026-02-05 11:47 ` "davecramer (@davecramer)" <[email protected]> 4 siblings, 0 replies; 6+ messages in thread From: davecramer (@davecramer) @ 2026-02-05 11:47 UTC (permalink / raw) To: pgjdbc/pgjdbc <[email protected]> (on pgjdbc/src/main/java/org/postgresql/core/v3/ConnectionFactoryImpl.java:224) ? ^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: [pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl @ 2026-02-05 11:48 ` "vlsi (@vlsi)" <[email protected]> 4 siblings, 0 replies; 6+ messages in thread From: vlsi (@vlsi) @ 2026-02-05 11:48 UTC (permalink / raw) To: pgjdbc/pgjdbc <[email protected]> (on pgjdbc/src/main/java/org/postgresql/core/v3/ConnectionFactoryImpl.java:224) I meant major vs minor ^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: [pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl @ 2026-02-05 12:24 ` "vlsi (@vlsi)" <[email protected]> 4 siblings, 0 replies; 6+ messages in thread From: vlsi (@vlsi) @ 2026-02-05 12:24 UTC (permalink / raw) To: pgjdbc/pgjdbc <[email protected]> (on pgjdbc/src/main/java/org/postgresql/core/v3/ConnectionFactoryImpl.java:224) Just for consistency, could we order maror/minor the same across both if branches? ^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: [pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl @ 2026-02-05 12:50 ` "davecramer (@davecramer)" <[email protected]> 4 siblings, 0 replies; 6+ messages in thread From: davecramer (@davecramer) @ 2026-02-05 12:50 UTC (permalink / raw) To: pgjdbc/pgjdbc <[email protected]> (on pgjdbc/src/main/java/org/postgresql/core/v3/ConnectionFactoryImpl.java:224) Yea, I thought that was weird too ^ permalink raw reply [nested|flat] 6+ messages in thread
end of thread, other threads:[~2026-02-05 12:50 UTC | newest] Thread overview: 6+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2026-02-05 11:20 [pgjdbc/pgjdbc] PR #3929: fix small issues in ConnectionFactoryImpl "davecramer (@davecramer)" <[email protected]> 2026-02-05 11:33 ` "vlsi (@vlsi)" <[email protected]> 2026-02-05 11:47 ` "davecramer (@davecramer)" <[email protected]> 2026-02-05 11:48 ` "vlsi (@vlsi)" <[email protected]> 2026-02-05 12:24 ` "vlsi (@vlsi)" <[email protected]> 2026-02-05 12:50 ` "davecramer (@davecramer)" <[email protected]>
This inbox is served by agora; see mirroring instructions for how to clone and mirror all data and code used for this inbox