Skip to content

Commit

Permalink
Remove sleep from Observability Interop Test binary now that its done…
Browse files Browse the repository at this point in the history
… in close() (grpc#9977)

After grpc#9972, the `sleep()` is done inside Observability `close()`, we can remove this `sleep()` in the Observability Interop test binary.
  • Loading branch information
stanley-cheung authored and larry-safran committed Mar 23, 2023
1 parent f8b4ac9 commit 0f51b9c
Showing 1 changed file with 0 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
import io.grpc.testing.integration.TestServiceClient;
import io.grpc.testing.integration.TestServiceServer;
import java.util.Arrays;
import java.util.concurrent.TimeUnit;

/**
* Combined interop client and server for observability testing.
Expand All @@ -47,11 +46,6 @@ public static void main(String[] args) throws Exception {
} else {
TestServiceServer.main(args);
}
// TODO(stanleycheung): remove this once the observability exporter plugin is able to
// gracefully flush observability data to cloud at shutdown
final int o11yCloseSleepSeconds = 65;
System.out.println("Sleeping " + o11yCloseSleepSeconds + " seconds before exiting");
Thread.sleep(TimeUnit.MILLISECONDS.convert(o11yCloseSleepSeconds, TimeUnit.SECONDS));
}
}

Expand Down

0 comments on commit 0f51b9c

Please sign in to comment.