Skip to content

Change buffered_reader stream to a non-default one#17106

Merged
sneaxiy merged 2 commits intoPaddlePaddle:developfrom
sneaxiy:fix_reader_default_stream
Apr 30, 2019
Merged

Change buffered_reader stream to a non-default one#17106
sneaxiy merged 2 commits intoPaddlePaddle:developfrom
sneaxiy:fix_reader_default_stream

Conversation

@sneaxiy
Copy link
Collaborator

@sneaxiy sneaxiy commented Apr 25, 2019

No description provided.

chengduoZH
chengduoZH previously approved these changes Apr 26, 2019
Copy link
Contributor

@chengduoZH chengduoZH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you change extra_time here?

Copy link
Collaborator Author

@sneaxiy sneaxiy Apr 28, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not know why retry_allocator_test fails randomly yet since CI is changed to run in parallel recently.

test=develop
@sneaxiy sneaxiy force-pushed the fix_reader_default_stream branch from 5bc4188 to 3d83a6f Compare April 28, 2019 03:28
Copy link
Contributor

@chengduoZH chengduoZH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent

@sneaxiy sneaxiy merged commit 08773b6 into PaddlePaddle:develop Apr 30, 2019
@sneaxiy sneaxiy deleted the fix_reader_default_stream branch October 17, 2019 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants