InfluxDB Logger Support

[dennypage]

@hubitrep

If I read this right, you seem to have gotten rid of the semaphore altogether. I have read elsewhere that state.* is not considered thread-safe and the app's event handler can get called from multiple threads. The alternative I've seen suggested is to use atomicState.* variables but, for those, writes go to the db directly. I suspect the performance might get even worse.

You are correct, the state map is not multi-thread safe. The semaphore isn't really gone, it's just externalized. The magic is on line 52. In Hubitat 2.2.9, they introduced the singleThreaded attribute for both apps and drivers, which provides externally managed thread exclusion. It is faster than atomicState and designed for just this kind of situation.

[Edit: singleThreaded attribute described here]