-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add initial cluster data version support #13400
Add initial cluster data version support #13400
Conversation
PR #13400: Size comparison from b7ff37e to 20dad18 Increases above 0.2%:
Increases (25 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
20dad18
to
f81b247
Compare
a9ed39a
to
fb9726f
Compare
PR #13400: Size comparison from 6ba06b2 to fb9726f Increases above 0.2%:
Increases (29 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (33 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
fb9726f
to
96a4524
Compare
PR #13400: Size comparison from 43a1e1a to 96a4524 Increases (21 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
Full report (23 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
96a4524
to
8523384
Compare
PR #13400: Size comparison from ddb7ce7 to 8523384 Increases above 0.2%:
Increases (29 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (33 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #13400: Size comparison from ddb7ce7 to 07462f0 Increases above 0.2%:
Increases (29 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (33 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #13400: Size comparison from ddb7ce7 to 9f5be31 Increases above 0.2%:
Increases (2 builds for linux, qpg)
Decreases (14 builds for cyw30739, efr32, k32w, linux, p6, qpg, telink)
Full report (16 builds for cyw30739, efr32, k32w, linux, p6, qpg, telink)
|
Where are the checks of the data version in the ReadRequest against the version currently present on the server, and omitting data if that matches? That presumably is the biggest reason we're doing this whole version feature, which is to optimize out traffic for reads/subscribes. |
DataVersionFilters has not yet been implemented in this PR, would happen in next PR, thanks |
PR #13400: Size comparison from ddb7ce7 to 0d52ab6 Increases above 0.2%:
Increases (4 builds for esp32, linux, qpg)
Decreases (20 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
Full report (23 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
|
/rebase |
0d52ab6
to
f7fa122
Compare
PR #13400: Size comparison from fb256c5 to f7fa122 Increases above 0.2%:
Increases (29 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (33 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
Need to integrate cluster data version storage with IM
Change overview
Update ReadHandler/Engine to retrieve version from ember, and process version in read client side.
Testing
Manually observe the data version change using IM read/write interaction.