Commit 5a3650a
[SNOW-3249917] JDBC removal Step 11a (part 2): Replicate HttpUtil, swap callers to replicated REST infrastructure
Replicate JDBC's HttpUtil as JdbcHttpUtil (renamed to avoid collision with
ingest's existing net.snowflake.ingest.utils.HttpUtil). Also replicate
SnowflakeMutableProxyRoutePlanner and AttributeEnhancingHttpRequestRetryHandler
as small helper classes.
Key changes:
- JdbcHttpUtil: verbatim replication of JDBC HttpUtil with import swaps.
SFTrustManager replaced with null (ingest does not use OCSP trust manager).
SFSSLConnectionSocketFactory -> IngestSSLConnectionSocketFactory.
SystemUtil.convertSystemPropertyToIntValue inlined.
SessionUtil.isNewRetryStrategyRequest inlined as static method.
Deprecated S3 proxy wrapper methods omitted (S3HttpUtil callable directly).
- RestRequest: all FQN net.snowflake.client.core.HttpUtil references replaced
with JdbcHttpUtil. SessionUtil.isNewRetryStrategyRequest replaced with
JdbcHttpUtil.isNewRetryStrategyRequest.
- TelemetryClient: HttpUtil.executeGeneralRequest and HttpUtil.getSocketTimeout
replaced with JdbcHttpUtil equivalents. SnowflakeSQLException import swapped
to ingest's replicated version.
- SnowflakeGCSClient: HttpUtil.getHttpClient, getHttpClientWithoutDecompression,
getSocketTimeout replaced with JdbcHttpUtil equivalents. Added toIngestKey
adapter to convert JDBC's HttpClientSettingsKey during transition.
- SnowflakeAzureClient: setSessionlessProxyForAzure import swapped from JDBC
HttpUtil to JdbcHttpUtil.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>1 parent 72e1169 commit 5a3650a
File tree
8 files changed
+1327
-23
lines changed- .plans
- src/main/java/net/snowflake/ingest
- connection/telemetry
- streaming/internal/fileTransferAgent
8 files changed
+1327
-23
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| 15 | + | |
| 16 | + | |
15 | 17 | | |
16 | 18 | | |
17 | 19 | | |
| |||
Lines changed: 4 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
| 14 | + | |
16 | 15 | | |
| 16 | + | |
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| |||
275 | 275 | | |
276 | 276 | | |
277 | 277 | | |
278 | | - | |
| 278 | + | |
279 | 279 | | |
280 | 280 | | |
281 | 281 | | |
282 | | - | |
| 282 | + | |
283 | 283 | | |
284 | 284 | | |
285 | 285 | | |
| |||
Lines changed: 32 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
0 commit comments