[SNOW-3249917] JDBC removal Step 11a: Replicate RestRequest and HTTP retry infrastructure (#1143)

* Update JDBC removal plan: add Step 11 (final cleanup)

Step 10c/10c2 done (remove SFSession). After those, 6 JDBC imports +
~70 FQN references remain. Split final cleanup into Step 11:

- 11a: Create HttpRequestHelper, replace 6 JDBC HTTP calls
- 11b: Remove 47 FQN SnowflakeSQLException from throws clauses
- 11c: Clean up remaining FQN refs (HttpClientSettingsKey, HttpProtocol,
  OCSPMode, SnowflakeUtil, setSessionlessProxyForAzure, AwsSdkGCPSigner)
- 11d: Demote JDBC to test scope + remove shade rules

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* [SNOW-3249917] JDBC removal Step 11a: Replicate RestRequest and HTTP retry infrastructure

Replicate the following JDBC classes (v3.25.1) verbatim into the ingest
fileTransferAgent package:

- ThrowingBiFunction (functional interface for retry callbacks)
- OCSPErrorCode (OCSP error code enum)
- SFOCSPException (OCSP exception)
- URLUtil (URL validation and request ID extraction)
- DecorrelatedJitterBackoff (already existed, now tracked)
- RetryContext (retry state holder)
- RetryContextManager (retry callback registration)
- HttpExecutingContext (HTTP request execution state)
- HttpExecutingContextBuilder (builder for HttpExecutingContext)
- SnowflakeUseDPoPNonceException (DPoP nonce error)
- RestRequest (1265-line HTTP request retry engine)

Import swaps applied per the JDBC removal plan. HttpUtil references kept
as FQN (net.snowflake.client.core.HttpUtil) pending future replication.
SessionUtil.isNewRetryStrategyRequest kept as FQN. SnowflakeUtil.logResponseDetails
inlined as a local method since it requires JDBC's SFLogger type.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: sfc-gh-ggeng

.plans/JDBC_REMOVAL_PLAN.md

@@ -278,8 +278,12 @@ is replaced.
 | Step 9c — Swap telemetry imports | ✅ Open | #1131 |
 | Step 10a — Replicate SFException, ExecTimeTelemetryData, etc. | ✅ Open | #1132 |
 | Step 10b — Swap SFException imports | ✅ Open | #1134 |
-| Step 10c — Remove SFSession/SFBaseSession | ⬜ TODO | — |
-| Step 10d — Demote JDBC to test scope | ⬜ TODO | — |
+| Step 10c — Remove SFSession from storage stack | ✅ Open | #1135 |
+| Step 10c2 — Remove SFSession from exceptions + telemetry | ✅ Open | #1136 |
+| Step 11a — Replace JDBC HTTP calls with HttpRequestHelper | ⬜ TODO | — |
+| Step 11b — Remove FQN SnowflakeSQLException from throws | ⬜ TODO | — |
+| Step 11c — Clean up remaining FQN JDBC references | ⬜ TODO | — |
+| Step 11d — Demote JDBC to test scope | ⬜ TODO | — |
 
 **Closed PRs:** #1117 (reverted 7b approach), #1122 (reverted 8c approach)
 **Other PRs:** #1118 (error/exception tests on master), #1133 (Maven retry config)
@@ -616,34 +620,74 @@ NOT swapped — they interact with JDBC's `RestRequest.executeWithRetries()`.
 
 ---
 
-### Step 10c — Remove SFSession/SFBaseSession ⬜ TODO
+### Step 10c — Remove SFSession from storage stack ✅ Open (PR #1135)
 
-SFSession/SFBaseSession are always null from ingest callers. Not feasible
-to replicate (1498+1404 lines, 156-class transitive closure = 40K lines).
-Need to remove these parameter types.
+**Done:** Remove SFSession/SFBaseSession parameters and dead session-based
+code from storage clients, interface, strategies, factory, agent, config.
+Session was always null from ingest callers. -336 lines.
 
 ---
 
-### Step 10d — Demote JDBC to test scope ⬜ TODO
+### Step 10c2 — Remove SFSession from exceptions + telemetry ✅ Open (PR #1136)
 
-Remaining 27 JDBC imports after Step 10b (all unreplicable due to massive
-dependency chains):
-- `SFSession`/`SFBaseSession` (15) — parameter types, always null
-- `HttpUtil` (2) — GCS client + TelemetryClient
-- `RestRequest` (1) — GCS client
-- `SnowflakeConnectionV1` (1) — TelemetryClient session path
-- `SnowflakeSQLException` (JDBC's, 1) — TelemetryClient
-- `ExecTimeTelemetryData`/`HttpResponseContextDto` (2) — GCS client
-  (replicated but interact with JDBC RestRequest)
-- IB `Telemetry`/`TelemetryField`/`TelemetryUtil` (3) —
-  interact with session.getTelemetryClient()
-- `SFSession` in `SnowflakeSQLLoggedException` (2) — parameter types
+**Done:** Remove SFSession/SFBaseSession from SnowflakeSQLLoggedException
+(all 15 constructors) and TelemetryClient (session-based code). Remove IB
+telemetry dead code. Update all callers (~12 files). -339 lines.
 
-Then:
+After 10c2: 6 JDBC imports remain + ~70 FQN JDBC references in throws/params.
+
+---
+
+### Step 11a — Replace JDBC HTTP calls with HttpRequestHelper ⬜ TODO
+
+Create `HttpRequestHelper` utility with retry logic (replaces JDBC's
+`RestRequest.executeWithRetries` and `HttpUtil.executeGeneralRequest`).
+Replace the 6 remaining JDBC imports:
+
+- `TelemetryClient`: replace `HttpUtil.executeGeneralRequest()` +
+  `SnowflakeSQLException` catch
+- `SnowflakeGCSClient`: replace `HttpUtil.getHttpClientWithoutDecompression()`,
+  `HttpUtil.getHttpClient()`, `RestRequest.executeWithRetries()`,
+  `HttpUtil.getSocketTimeout()`. Remove `ExecTimeTelemetryData`,
+  `HttpResponseContextDto`, `RestRequest` imports.
+
+---
+
+### Step 11b — Remove FQN SnowflakeSQLException from throws ⬜ TODO
+
+Mechanical removal of `, net.snowflake.client.jdbc.SnowflakeSQLException`
+from ~47 throws clauses across all replicated storage clients, interface,
+strategies, factory, and GCS client.
+
+---
+
+### Step 11c — Clean up remaining FQN JDBC references ⬜ TODO
+
+Swap remaining FQN JDBC type references to ingest versions:
+- `net.snowflake.client.core.HttpClientSettingsKey` → `HttpClientSettingsKey`
+  (same package, 8 occurrences in S3HttpUtil, SnowflakeFileTransferAgent,
+  SnowflakeGCSClient, SnowflakeStorageClient)
+- `net.snowflake.client.core.HttpProtocol` → `HttpProtocol` (same package,
+  1 occurrence in S3HttpUtil)
+- `net.snowflake.client.core.OCSPMode` → `net.snowflake.ingest.utils.OCSPMode`
+  (2 occurrences in SnowflakeFileTransferAgent)
+- `net.snowflake.client.jdbc.SnowflakeUtil.convertProxyPropertiesToHttpClientKey`
+  → `StorageClientUtil.convertProxyPropertiesToHttpClientKey` (2 occurrences
+  in SnowflakeFileTransferAgent)
+- `static import net.snowflake.client.core.HttpUtil.setSessionlessProxyForAzure`
+  → replicate method in StorageClientUtil (4 occurrences in SnowflakeAzureClient)
+- `net.snowflake.client.jdbc.cloud.storage.AwsSdkGCPSigner` — JDBC class
+  reference in GCSAccessStrategyAwsSdk (string constant + class reference,
+  3 occurrences)
+
+---
+
+### Step 11d — Demote JDBC to test scope ⬜ TODO
+
+After all FQN references are cleaned up:
 1. Demote `snowflake-jdbc-thin` to `test` scope in `pom.xml`
 2. Remove JDBC shade relocation rules from Maven Shade plugin
-3. Remove `snowflake-jdbc-thin` from `public_pom.xml`
-4. Run full test suite
+3. Run full test suite
 
 ---
 

src/main/java/net/snowflake/ingest/streaming/internal/fileTransferAgent/DecorrelatedJitterBackoff.java

@@ -0,0 +1,39 @@
+/*
+ * Replicated from snowflake-jdbc (v3.25.1)
+ * Source: https://github.com/snowflakedb/snowflake-jdbc/blob/v3.25.1/src/main/java/net/snowflake/client/util/DecorrelatedJitterBackoff.java
+ *
+ * Permitted differences: package.
+ */
+package net.snowflake.ingest.streaming.internal.fileTransferAgent;
+
+import java.util.concurrent.ThreadLocalRandom;
+
+/**
+ * Decorrelated Jitter backoff
+ *
+ * <p>https://aws.amazon.com/blogs/architecture/exponential-backoff-and-jitter/
+ */
+public class DecorrelatedJitterBackoff {
+  private final long base;
+  private final long cap;
+
+  public DecorrelatedJitterBackoff(long base, long cap) {
+    this.base = base;
+    this.cap = cap;
+  }
+
+  public long nextSleepTime(long sleep) {
+    long correctedSleep = sleep <= base ? base + 1 : sleep;
+    return Math.min(cap, ThreadLocalRandom.current().nextLong(base, correctedSleep));
+  }
+
+  public long getJitterForLogin(long currentTime) {
+    double multiplicationFactor = chooseRandom(-1, 1);
+    long jitter = (long) (multiplicationFactor * currentTime * 0.5);
+    return jitter;
+  }
+
+  public double chooseRandom(double min, double max) {
+    return min + (Math.random() * (max - min));
+  }
+}