[SNOW-3249917] JDBC removal Step 10a: Replicate remaining small JDBC classes#1132
Merged
sfc-gh-ggeng merged 1 commit intomasterfrom Apr 2, 2026
Merged
Conversation
6b14c9b to
954d1b8
Compare
94fd850 to
5943e3a
Compare
954d1b8 to
87137be
Compare
5943e3a to
c9fad98
Compare
9f9cff1 to
693cd81
Compare
08529fb to
783bfbc
Compare
76f2f03 to
96d9968
Compare
783bfbc to
6f83093
Compare
96d9968 to
919ec03
Compare
6f83093 to
53fa30c
Compare
919ec03 to
bc98550
Compare
53fa30c to
71eab2a
Compare
bc98550 to
279c755
Compare
71eab2a to
1cb88ff
Compare
sfc-gh-alhuang
approved these changes
Apr 2, 2026
279c755 to
939fdf3
Compare
1cb88ff to
63cdb21
Compare
Base automatically changed from
jdbc-removal-step9c-swap-telemetry-imports
to
master
April 2, 2026 09:06
…classes Verbatim replication of remaining JDBC classes with self-contained dependency trees: - SFException (127 lines) — exception type used by SnowflakeSQLException and TelemetryEvent. Source: net.snowflake.client.core.SFException - HttpResponseContextDto (54 lines) — HTTP response context used by SnowflakeGCSClient. Source: net.snowflake.client.core.HttpResponseContextDto - TimeMeasurement (53 lines) — epoch microsecond timer used by ExecTimeTelemetryData. Source: net.snowflake.client.util.TimeMeasurement - ExecTimeTelemetryData (199 lines) — execution time telemetry used by SnowflakeGCSClient. Source: net.snowflake.client.core.ExecTimeTelemetryData - ThrowingCallable (9 lines) — functional interface used by HttpUtil. Source: net.snowflake.client.util.ThrowingCallable - StorageClientUtil: added getEpochTimeInMicroSeconds (from SnowflakeUtil, used by TimeMeasurement) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
939fdf3 to
99b0353
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verbatim replication of remaining JDBC classes with self-contained dependency trees:
SFException(127 lines) — exception type used bySnowflakeSQLExceptionandTelemetryEventnet.snowflake.client.core.SFExceptionHttpResponseContextDto(54 lines) — HTTP response context used bySnowflakeGCSClientnet.snowflake.client.core.HttpResponseContextDtoTimeMeasurement(53 lines) — epoch microsecond timer used byExecTimeTelemetryDatanet.snowflake.client.util.TimeMeasurementExecTimeTelemetryData(199 lines) — execution time telemetry used bySnowflakeGCSClientnet.snowflake.client.core.ExecTimeTelemetryDataThrowingCallable(9 lines) — functional interface used byHttpUtilnet.snowflake.client.util.ThrowingCallableStorageClientUtil: addedgetEpochTimeInMicroSeconds(fromSnowflakeUtil)Diff report
All classes replicated verbatim. Permitted differences:
@SnowflakeJdbcInternalApiremovedSnowflakeUtil.getEpochTimeInMicroSeconds→StorageClientUtil.getEpochTimeInMicroSecondsSnowflakeUtil.isNullOrEmpty→StorageClientUtil.isNullOrEmptySFLogger/SFLoggerFactory→ ingest versionsErrorCode/ResourceBundleManager/TelemetryService/TimeMeasurement→ same packageUnexpected differences: NONE
Test plan
mvn compiler:compilepasses🤖 Generated with Claude Code