benWize commented on a change in pull request #15648:
URL: https://github.com/apache/beam/pull/15648#discussion_r728539578



##########
File path: 
runners/core-construction-java/src/test/java/org/apache/beam/runners/core/construction/CommonCoderTest.java
##########
@@ -514,20 +511,4 @@ private void verifyDecodedValue(CommonCoder coder, Object 
expectedValue, Object
       throw new IllegalStateException("Unknown coder URN: " + coder.getUrn());
     }
   }
-
-  /**
-   * Utility for adding new entries to the common coder spec -- prints the 
serialized bytes of the
-   * given value in the given context using JSON-escaped strings.
-   */
-  private static <T> String jsonByteString(Coder<T> coder, T value, Context 
context)

Review comment:
       Yes, done!




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to