Message-ID: From: "vlsi (@vlsi)" To: "pgjdbc/pgjdbc" Date: Wed, 09 Apr 2025 17:53:07 +0000 Subject: Re: [pgjdbc/pgjdbc] PR #3568: use pgType instead of internal name to preserve the schema In-Reply-To: References: List-Id: X-GitHub-Author-Login: vlsi X-GitHub-Comment-Id: 2035856811 X-GitHub-Comment-Type: review_comment X-GitHub-Commit: fdef48c6ec57cd9217d8cefacbe7e60cd230b1e6 X-GitHub-Issue: 3568 X-GitHub-Line: 548 X-GitHub-Path: pgjdbc/src/main/java/org/postgresql/jdbc/TypeInfoCache.java X-GitHub-Repo: pgjdbc/pgjdbc X-GitHub-Type: review_comment X-GitHub-Url: https://github.com/pgjdbc/pgjdbc/pull/3568#discussion_r2035856811 Content-Type: text/plain; charset=utf-8 (on pgjdbc/src/main/java/org/postgresql/jdbc/TypeInfoCache.java:548) What is the reason to put a truncated type name into the cache? If someone calls the function with a full name like `"public"."typename"`, then it would always end up with `getOidStatement` which is unwanted.