leezu commented on a change in pull request #17797: Skip
test_multi_worker_dataloader_release_pool
URL: https://github.com/apache/incubator-mxnet/pull/17797#discussion_r392607787
##########
File path: tests/python/unittest/test_gluon_data.py
##########
@@ -269,6 +269,12 @@ def test_multi_worker_dataloader_release_pool():
if os.name == 'nt':
print('Skip for windows since spawn on windows is too expensive.')
return
+
+ from sys import platform
+ if platform == 'darwin':
+ print('Skip for MacOS due to flaky failures')
Review comment:
@haojin2 can you merge the suggested change or something similar to address
Marco's concern?
----------------------------------------------------------------
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]
With regards,
Apache Git Services
