-
Notifications
You must be signed in to change notification settings - Fork 178
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deps: remove client-core dependency from bigtable-hbase-2.x #3620
Conversation
BigtableInstanceName bigtableInstanceName = | ||
new BigtableInstanceName( | ||
bigtableSnapshotClusterName = | ||
ClusterName.of( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use NameUtil from veneer
@@ -86,7 +85,7 @@ public abstract class BigtableAsyncAdmin implements AsyncAdmin { | |||
private final BigtableHBaseSettings settings; | |||
private final CommonConnection asyncConnection; | |||
private final String bigtableInstanceName; | |||
private BigtableClusterName bigtableSnapshotClusterName; | |||
private String bigtableSnapshotClusterName; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
snapshotClusterId
String clusterId = | ||
configuration.get(BigtableOptionsFactory.BIGTABLE_SNAPSHOT_CLUSTER_ID_KEY, null); | ||
if (clusterId != null) { | ||
BigtableInstanceName bigtableInstanceName = | ||
new BigtableInstanceName( | ||
asyncConnection.getBigtableSettings().getProjectId(), | ||
asyncConnection.getBigtableSettings().getInstanceId()); | ||
bigtableSnapshotClusterName = bigtableInstanceName.toClusterName(clusterId); | ||
bigtableSnapshotClusterName = clusterId; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you just assign it?
@@ -517,7 +512,7 @@ private static <T> List<T> filter(Collection<T> r, Predicate<T> predicate) { | |||
return r.stream().filter(predicate).collect(Collectors.toList()); | |||
} | |||
|
|||
private synchronized BigtableClusterName getBackupClusterName() { | |||
private synchronized String getBackupClusterName() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
getBackupClusterId()
Name == is the fully qualified name: projects/my-project/instances/my-instance/clusters/my-clsuter
Id = is the short identifier: my-cluster
@@ -86,7 +84,7 @@ public abstract class BigtableAsyncAdmin implements AsyncAdmin { | |||
private final BigtableHBaseSettings settings; | |||
private final CommonConnection asyncConnection; | |||
private final String bigtableInstanceName; | |||
private BigtableClusterName bigtableSnapshotClusterName; | |||
private String bigtableSnapshotClusterId; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit. I dont think bigtable prefix is necessary here...I think its inferred from the context
String clusterId = | ||
configuration.get(BigtableOptionsFactory.BIGTABLE_SNAPSHOT_CLUSTER_ID_KEY, null); | ||
if (clusterId != null) { | ||
BigtableInstanceName bigtableInstanceName = | ||
new BigtableInstanceName( | ||
asyncConnection.getBigtableSettings().getProjectId(), | ||
asyncConnection.getBigtableSettings().getInstanceId()); | ||
bigtableSnapshotClusterName = bigtableInstanceName.toClusterName(clusterId); | ||
bigtableSnapshotClusterId = clusterId; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think the if statement is relevant any more
.../bigtable-hbase-2.x/src/main/java/com/google/cloud/bigtable/hbase2_x/BigtableAsyncAdmin.java
Outdated
Show resolved
Hide resolved
🤖 I have created a release *beep* *boop* --- ## [2.3.0](v2.2.0...v2.3.0) (2022-05-18) ### Features * add enableSnappy flag to import snapshot pipeline and select th… ([#3586](#3586)) ([8c1854d](8c1854d)) ### Dependencies * remove client-core dependency from bigtable-hbase-2.x ([#3620](#3620)) ([413a2fd](413a2fd)) ### Documentation * Update README to use --enableSnappy flag to import snappy compre… ([#3623](#3623)) ([ea73be9](ea73be9)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
No description provided.