-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Fix race condition in unittest by pytest temp_dir fixtures #18323
Conversation
Hey @zhreshold , Thanks for submitting the PR
CI supported jobs: [windows-gpu, centos-gpu, sanity, windows-cpu, miscellaneous, unix-cpu, clang, edge, website, unix-gpu, centos-cpu] Note: |
Could you clarify where the race occurs? |
If multiple tests want to create the rec file at the same time, the rec file may be corrupted |
Would creating the rec-file at a temporary path then solve the issue while preserving parallelization? You can take the |
@mxnet-bot run ci [centos-cpu, unix-cpu] |
Jenkins CI successfully triggered : [centos-cpu, unix-cpu] |
prepare_record() | ||
dataset = gluon.data.vision.ImageFolderDataset('data/test_images') | ||
def test_image_folder_dataset(prepare_record): | ||
print(prepare_record[:-8]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
Fix #18322
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments