y1chi commented on a change in pull request #12709:
URL: https://github.com/apache/beam/pull/12709#discussion_r483289536
##########
File path:
sdks/python/apache_beam/testing/benchmarks/nexmark/nexmark_launcher.py
##########
@@ -67,18 +67,20 @@
import logging
import sys
import uuid
-
-from google.cloud import pubsub
+from time import sleep
Review comment:
since you wouldn't have to import each function you want to use from
time module. It's not a big deal but I believe calling time.sleep() and
time.time() will be concise enough.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
