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

Dynatrace file-based configuration slow on macOS #3035

Closed
pirgeo opened this issue Feb 22, 2022 · 0 comments · Fixed by #3065
Closed

Dynatrace file-based configuration slow on macOS #3035

pirgeo opened this issue Feb 22, 2022 · 0 comments · Fixed by #3065
Labels
bug A general bug registry: dynatrace A Dynatrace Registry related issue
Milestone

Comments

@pirgeo
Copy link
Contributor

pirgeo commented Feb 22, 2022

Thanks to @jonatan-ivanov for pointing out a problem with the WatchService based approach on macOS. A native implementation seems to be missing for the macOS file systems (at least in some JDKs), leading to a fallback to a polling implementation that checks the filesystem for changes every 10s. This is consistent with what @jonatan-ivanov reported in #2916. We are currently looking into this.

Original comment for context:

@pirgeo I think there are some issues with the Dynatrace file poller, it seems it takes a lot of time on macOS for the file poller to detect the changes.

The tests in this PR were failing for me on my mac (OS X 12.1, x86_64): see this report.
I replaced Thread::sleep with awaitility it takes almost 10 seconds on my machine to see the file changes (instead of 10ms that you had). Could you please look into it if anyone else has this issue before?

My changes: jonatan-ivanov@f4c1c42 (they are unrelated since the build was failing for me at the first place).

Originally posted by @jonatan-ivanov in #2916 (comment)

@shakuzen shakuzen added registry: dynatrace A Dynatrace Registry related issue bug A general bug labels Feb 22, 2022
@shakuzen shakuzen added this to the 1.9 backlog milestone Feb 22, 2022
@shakuzen shakuzen modified the milestones: 1.9 backlog, 1.9.0-M4 Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A general bug registry: dynatrace A Dynatrace Registry related issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants