fix: change toString to value in TopicStatsClient (#210)

googleapis · Aug 19, 2020 · 30f0b10 · 30f0b10
1 parent a4ca349
commit 30f0b10
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/...cloud-pubsublite/src/main/java/com/google/cloud/pubsublite/internal/TopicStatsClient.java b/...cloud-pubsublite/src/main/java/com/google/cloud/pubsublite/internal/TopicStatsClient.java
@@ -51,5 +51,6 @@ ApiFuture<ComputeMessageStatsResponse> computeMessageStats(
    *
    * @throws StatusException on a failure to properly terminate.
    */
+  @Override
   void close() throws StatusException;
 }
diff --git a/...d-pubsublite/src/main/java/com/google/cloud/pubsublite/internal/TopicStatsClientImpl.java b/...d-pubsublite/src/main/java/com/google/cloud/pubsublite/internal/TopicStatsClientImpl.java
@@ -110,7 +110,7 @@ public ApiFuture<ComputeMessageStatsResponse> computeMessageStats(
         () -> {
           return stub.computeMessageStats(
               ComputeMessageStatsRequest.newBuilder()
-                  .setTopic(path.toString())
+                  .setTopic(path.value())
                   .setPartition(partition.value())
                   .setStartCursor(Cursor.newBuilder().setOffset(start.value()).build())
                   .setEndCursor(Cursor.newBuilder().setOffset(end.value()).build())

diff --git a/...bsublite/src/test/java/com/google/cloud/pubsublite/internal/TopicStatsClientImplTest.java b/...bsublite/src/test/java/com/google/cloud/pubsublite/internal/TopicStatsClientImplTest.java
@@ -96,7 +96,7 @@ private static Offset end() {
 
   private static ComputeMessageStatsRequest request() {
     return ComputeMessageStatsRequest.newBuilder()
-        .setTopic(path().toString())
+        .setTopic(path().value())
         .setPartition(partition().value())
         .setStartCursor(Cursor.newBuilder().setOffset(start().value()).build())
         .setEndCursor(Cursor.newBuilder().setOffset(end().value()).build())