Github user aviyoop commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/207#discussion_r151419226
--- Diff: aria/utils/threading.py ---
@@ -161,11 +242,7 @@ def close(self):
self._workers = None
def drain(self):
- """
- Blocks until all current tasks finish execution, but leaves the
worker threads alive.
- """
-
- self._tasks.join() # oddly, the API does not support a timeout
parameter
+ self._tasks.join() # oddly, the API does not support a timeout
parameter
--- End diff --
In general, according to PEP8, `Inline comments should be separated by at
least two spaces from the statement`.
https://www.python.org/dev/peps/pep-0008/#id32
---
