Replace truetime with networktime

[stoyicker]

Doesn't lend itself to much of a split I'm afraid.

[stoyicker]

TODO Mark as ready when we release dropping the forks

[paulinaaniola]

I see that it's a draft but when it will be ready for review could you please elaborate a bit more about this change? I see some changes to the event producer and I don’t know where it came from.

[stoyicker]

I see that it's a draft but when it will be ready for review could you please elaborate a bit more about this change? I see some changes to the event producer and I don’t know where it came from.

You may consider this ready for code review now. The reason it's a draft is to prevent it from being merged before we've released dropping MQA, but no code changes are planned.

The changes you see in EventProducer are replacing truetime with networktime. The sleep block is to ensure initialization of the time before it gets queried. TrueTime also had this issue (it's more of a design thing really), which caused crashes. What I've done is prevent queries until they're not null in all sdks to avoid the problem, so you can see the same in Player, which makes us no longer need the custom application class that we used to get around the problem when we used truetime. This guarantees that even if an app uses either Player or EventProducer directly without having done the fencing themselves, we are not an issue for them. So there should be nothing to worry about when it comes to that.