Skip to content

Upgrade work queue #38335

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 37 commits into from
Dec 23, 2021
Merged

Upgrade work queue #38335

merged 37 commits into from
Dec 23, 2021

Conversation

@liutiexing
Copy link
Contributor

@liutiexing liutiexing commented Dec 21, 2021 •
edited
Loading

PR types

New features

PR changes

Others

Describe

Upgrade EventsWaiter

  1. Add UnregisterEvent() interface.
  2. Add LevelTriggered/EdgeTriggered choices for RegisterEvent().
  3. Add Clear() interface.
  4. Add detached option.
  5. Split WorkQueue files into its directory

liutiexing and others added 29 commits September 22, 2021 20:09
@liutiexing
add align for WorkQueue
8744ba7
@liutiexing
Merge branch 'develop' of https://github.com/liutiexing/Paddle into d…
4759bc8 
…evelop
@liutiexing
add spinlock
6f00ace
@liutiexing
merge develop
2d6f1cf
@liutiexing
Merge branch 'develop' of https://github.com/liutiexing/Paddle into d…
f5099be 
…evelop
@liutiexing
merge develop
54aa332
@liutiexing
merge
1d1bd82
@liutiexing
Merge remote-tracking branch 'upstream/develop' into develop
dfbf3e4
@liutiexing
Merge remote-tracking branch 'upstream/develop' into develop
a5392b3
@liutiexing
Add EventsWaiter
e206173
@liutiexing
Revert "Add EventsWaiter"
0a3dcd9 
This reverts commit e206173.
@liutiexing
Merge remote-tracking branch 'upstream/develop' into develop
4689bb5
@liutiexing
Merge remote-tracking branch 'upstream/develop' into develop
0cec99a
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
481c4fa
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
83db84e
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
7010e0d
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
ec2a363
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
90a59ec
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
1445bbe
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
a2c74ab
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
1c09b4e
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
cb8cf7d
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
cf0dcd6
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
2f95801
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
14bec1b
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
8a5f7af
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
f0a5915
@liutiexing
Merge branch 'PaddlePaddle:develop' into develop
0fe35aa
update EventsWater
3876d20
@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

Aurelius84
Aurelius84 approved these changes Dec 23, 2021
View reviewed changes
Copy link
Contributor

@Aurelius84 Aurelius84 left a comment

Choose a reason for hiding this comment

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

LGTM

zhiqiu
zhiqiu approved these changes Dec 23, 2021
View reviewed changes
Copy link
Contributor

@zhiqiu zhiqiu left a comment

Choose a reason for hiding this comment

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

LGTM

@Aurelius84 Aurelius84 merged commit 198d11b into PaddlePaddle:develop Dec 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zhiqiu zhiqiu zhiqiu approved these changes

@Aurelius84 Aurelius84 Aurelius84 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@liutiexing @zhiqiu @Aurelius84