Abacn commented on code in PR #29277:
URL: https://github.com/apache/beam/pull/29277#discussion_r1397372778


##########
sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/bigtable/changestreams/it/BigtableChangeStreamIT.java:
##########
@@ -145,13 +145,16 @@ public static void afterClass() {
 
   @Test
   public void testReadBigtableChangeStream() throws InterruptedException {
+    // Sleep to ensure clock discrepancy between client and server don't 
affect the start time.
+    Thread.sleep(500);

Review Comment:
   Could we just relax the start time and stop time, and create a transient 
table so that if other test runs concurrently it won't interfere each other?



-- 
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