Message-ID: From: "vlsi (@vlsi)" To: "pgjdbc/pgjdbc" Date: Wed, 12 Feb 2025 11:21:27 +0000 Subject: Re: [pgjdbc/pgjdbc] PR #3510: use in row values instead of union all In-Reply-To: References: List-Id: X-GitHub-Author-Login: vlsi X-GitHub-Comment-Id: 1952460346 X-GitHub-Comment-Type: review_comment X-GitHub-Commit: 54177351024badd76694127a97f19907f57236b9 X-GitHub-Issue: 3510 X-GitHub-Line: 252 X-GitHub-Path: pgjdbc/src/main/java/org/postgresql/jdbc/PgResultSetMetaData.java X-GitHub-Repo: pgjdbc/pgjdbc X-GitHub-Type: review_comment X-GitHub-Url: https://github.com/pgjdbc/pgjdbc/pull/3510#discussion_r1952460346 Content-Type: text/plain; charset=utf-8 (on pgjdbc/src/main/java/org/postgresql/jdbc/PgResultSetMetaData.java:252) Why removing this? Previously, `fieldInfoFetched` was set to `true` in case metadata for all the fields was known. I guess it was there to prevent duplicate queries from `fetchFieldMetaData()`