Skip to content
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.

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

Disable further FileWatching tests #47920

Merged
merged 1 commit into from
Jan 2, 2023
Merged

Disable further FileWatching tests #47920

merged 1 commit into from
Jan 2, 2023

Conversation

wegank
Copy link
Contributor

@wegank wegank commented Dec 18, 2022

Generalizing #46497 to all macOS platforms, due to #46185 (comment) from NixOS/nixpkgs#205584.

@brenhinkeller brenhinkeller added test This change adds or pertains to unit tests filesystem Underlying file system and functions that use it labels Dec 18, 2022
@vtjnash
Copy link
Sponsor Member

vtjnash commented Dec 20, 2022

Possibly relatedly, libuv appears to explicitly request coalescing of events that occurred within 50ms of the first event (which may coalesce even more if the thread scheduling is delayed) according to https://github.com/libuv/libuv/blob/a3de1384c304553886c4e74dc82d9356df732cf8/src/unix/fsevents.c#L345

@KristofferC KristofferC merged commit 16f4f05 into JuliaLang:master Jan 2, 2023
@wegank wegank deleted the darwin-tests branch January 2, 2023 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
filesystem Underlying file system and functions that use it test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants