-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Description
Spanner JDBC driver is currently dependent on cloud storage. If I'm not mistaken, the only reason for that is the single convenience helper class to pull credentials from GCS.
This requires apps that use spanner but don't use cloud storage require this dependency, even if they're not storing credentials in GCS. Additionally, for apps in monorepo that use the same 3rdparty dependency versions, it makes it much harder to update just spanner/spanner-jdbc as it requires upgrading cloud storage as well which implies updating other apps.
I think that this class should be provided in cloud storage package, not spanner - especially given that is should be relatively easy to refactor out of spanner code.
Would it be possible to remove cloud storage dependency from spanner jdbc lib?