Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Close watcher after event is received. #1081

Merged
merged 1 commit into from
Mar 24, 2017
Merged

Close watcher after event is received. #1081

merged 1 commit into from
Mar 24, 2017

Conversation

govint
Copy link
Contributor

@govint govint commented Mar 24, 2017

Fixed and stress test shows no watcher creation errors and lsof has no fiiles opened for inotify.

@govint
Copy link
Contributor Author

govint commented Mar 24, 2017

Fixes #1080

pdhamdhere
pdhamdhere approved these changes Mar 24, 2017
View reviewed changes
Copy link
Contributor

@pdhamdhere pdhamdhere left a comment

Choose a reason for hiding this comment

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

Good catch! LGTM.

@msterin msterin merged commit f9eed3c into master Mar 24, 2017
@govint govint deleted the close-watcher branch March 25, 2017 03:25
@govint govint mentioned this pull request Mar 25, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@msterin msterin msterin approved these changes

@pdhamdhere pdhamdhere pdhamdhere approved these changes

Assignees
No one assigned
Labels
cla-not-required
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@govint @msterin @pdhamdhere @vmwclabot