chore: refactor parseAndRemovePartnerTokenProperty()

keshavdandeva · keshavdandeva · commit e25caf49996f · 2026-02-18T13:29:43.000Z
diff --git a/google-cloud-bigquery-jdbc/src/main/java/com/google/cloud/bigquery/jdbc/BigQueryJdbcUrlUtility.java b/google-cloud-bigquery-jdbc/src/main/java/com/google/cloud/bigquery/jdbc/BigQueryJdbcUrlUtility.java
@@ -797,12 +797,9 @@ private static String parseAndRemovePartnerTokenProperty(
 
     if (matcher.find()) {
       String content = matcher.group(1).trim();
+      urlBuilder.delete(matcher.start(), matcher.end());
       if (content.toUpperCase().startsWith("GPN:")) {
-        urlBuilder.delete(matcher.start(), matcher.end());
         return " (" + content + ")";
-      } else {
-        urlBuilder.delete(matcher.start(), matcher.end());
-        return null;
       }
     }
     return null;

Original file line number	Diff line number	Diff line change
`@@ -797,12 +797,9 @@ private static String parseAndRemovePartnerTokenProperty(`
`797`	`797`
`798`	`798`	`if (matcher.find()) {`
`799`	`799`	`String content = matcher.group(1).trim();`
	`800`	`+ urlBuilder.delete(matcher.start(), matcher.end());`
`800`	`801`	`if (content.toUpperCase().startsWith("GPN:")) {`
`801`		`- urlBuilder.delete(matcher.start(), matcher.end());`
`802`	`802`	`return " (" + content + ")";`
`803`		`- } else {`
`804`		`- urlBuilder.delete(matcher.start(), matcher.end());`
`805`		`- return null;`
`806`	`803`	`}`
`807`	`804`	`}`
`808`	`805`	`return null;`