influxdb pooling: add ability to pool db writes #27
Conversation
schulzad
commented
Oct 17, 2017
schulzad
commented
- fix for influxDB write error. 'null' is not a valid number for value= field
- Enabled by default. This handles only the softPolling() events.
- Misc cleanup and handle possibility of a null uptime
|
look like #20 might also fix the null value error. |
schulzad
force-pushed
the
influxdb-pooling
branch
from
704ae2d to
edd060b
Compare
refactor parsing code out into it's own function. in the softpoll build all the data that will be sent to influxdb and post it all at once.
fix for influxDB write error. 'null' is not a valid number for value= field
schulzad
force-pushed
the
influxdb-pooling
branch
from
afa310f to
5614e06
Compare
schulzad
force-pushed
the
influxdb-pooling
branch
from
cc56493 to
e25fecb
Compare
|
Is this being maintained anymore? If not, can you add me to this repo or fork the influxdb stuff to another repo so we can get some of these changes in? |
|
@codersaur I'm one of the SmartThings developers (as is @schulzad). Several of the core team use this integration and really enjoy it. This patch is fairly important for correct operations with a large number of devices as otherwise the softPoll generates an HubAction for each configured device. If a large number of things are configured, this results in dropped requests as the cloud has a max queue depth for LAN messages to the hub. These errors are not exposed directly to users as the queue depth restriction is handled by a different edge service managing the connections with the hub but you can see a pretty clear 5 minute spike of errors for my home setup due to the softPoll capping out the LAN queue. This can also affect other LAN automations that may be doing something more important than syncing data to influx. This PR collapses down all of the requests that would happen on softPoll down to a single HTTP request which keeps this from being a problem. Please let us know if you have any questions.
|
influxdb pooling: add ability to pool db writes codersaur#27
|
Did you guys think about creating an active fork or something? Seems abandoned here. |
Unfortunately the way the licensing is setup, this cannot be forked and maintained. At least not officially. |
