SmartThingsCommunity / SmartThingsEdgeDrivers

Apache License 2.0
269 stars 463 forks source link

Remove state cache usage from devices to reduce datastore size #1671

Closed cjswedes closed 1 month ago

cjswedes commented 1 month ago

There are some current usages of the state cache in the driver, but the impact seems minimal.

Similar to #1670

@dljsjr it would be great if you could do a test with this on your setup prior to merging. Also, looking for your input on the existing state cache optimizations that will be lost with this change. All usages should be in this diff.

github-actions[bot] commented 1 month ago

Channel deleted.

github-actions[bot] commented 1 month ago

Test Results

   63 files  ±0    396 suites  ±0   0s :stopwatch: ±0s 1 937 tests ±0  1 937 :white_check_mark: ±0  0 :zzz: ±0  0 :x: ±0  3 361 runs  ±0  3 361 :white_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit d23f1a65. ± Comparison against base commit 96f5474e.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

Minimum allowed coverage is 90%

Generated by :monkey: cobertura-action against 413f395ffc2bed22fa4ded8eece465200a53b838

github-actions[bot] commented 1 month ago

Invitation URL: https://bestow-regional.api.smartthings.com/invite/akMXaNeVoB2b