-
Notifications
You must be signed in to change notification settings - Fork 88
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
Is it possible to use multiple NTP hosts? #72
Comments
What's your use case? You don't want to overload stratum1/2 servers so ideally you'd just use |
@Reflejo thanks for the response 🙂. In our case, I want to use our Datadog pool:
Should I just set it to |
I don't think a) Why are you using that one pool for the client? |
Hey @Reflejo 👋. We work on logging and tracing SDK. Our goal is to synchronise time of events in a distributed environment (where tracing span starts on the mobile client and propagates through the stack of instrumented backend servers). Because our backend agents use NTP, we want to also use it on the mobile clients, to reduce temporal misalignments.
In theory, there should be no difference depending on which NTP pool I pick, but because our backend agents default to Datadog NTP, I consider it as a good configuration for the mobile SDK as well.
I'd expect that the library picks one of those hosts randomly and performs NTP sync with it. Currently, I implemented this logic in the SDK, and I just randomly set one of these 4 hosts as the Does it make sense 🙂? I'm not the expert on NTP topic and I don't know much of it details. |
Hello Lyft 🙂👋
I'm stepping across this project, as we're looking for NTP library. It looks great and does what we need with minimum code 🚀. Although, I'm wondering if it's possible to specify multiple NTP hosts?
Kronos Android supports it:
but the iOS version seems to only accept single host:
The text was updated successfully, but these errors were encountered: