shingjan commented on a change in pull request #8492:
URL: https://github.com/apache/tvm/pull/8492#discussion_r672729908



##########
File path: python/tvm/autotvm/tuner/xgboost_cost_model.py
##########
@@ -161,7 +162,8 @@ def _reset_pool(self, space, target, task):
         _extract_space = space
         _extract_target = target
         _extract_task = task
-        self.pool = multiprocessing.Pool(self.num_threads)
+        self.pool = popen_pool.PopenPoolExecutor(self.num_threads)

Review comment:
       `multiprocessing.pool()` seems to create a process pool instead of a 
thread pool. Hence the name of the argument passed here `self.num_threads` may 
not be the most accurate. Is that the case?




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