-
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
Serialize ContentApp SupportedCluster set #34144
Serialize ContentApp SupportedCluster set #34144
Conversation
PR #34144: Size comparison from d033f8b to 5515a49 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
5515a49
to
220abf8
Compare
PR #34144: Size comparison from 102faca to 220abf8 Full report (8 builds for cc32xx, mbed, qpg, stm32, tizen)
|
220abf8
to
1665f45
Compare
PR #34144: Size comparison from a8fe0cf to 1665f45 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
1665f45
to
6edb6c6
Compare
PR #34144: Size comparison from ca57b59 to 6edb6c6 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34144: Size comparison from ca57b59 to f8a663c Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34144: Size comparison from 6e31453 to 578a75c Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34144: Size comparison from 8786012 to 3d13852 Full report (51 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, tizen)
|
PR #34144: Size comparison from 8786012 to 28764b8 Full report (4 builds for cc32xx, mbed, stm32)
|
PR #34144: Size comparison from 8786012 to a44158f Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34144: Size comparison from 90310f2 to 6010a5f Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34144: Size comparison from 9306418 to e0b51d9 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34144: Size comparison from 8ebe19d to b56e6fa Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
* Serialize ContentApp SupportedCluster set * Restyled by google-java-format * Update EndpointsDataStore.java * Restyled by google-java-format --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Lazar Kovacic <lkovacic@amazon.com>
[Problem]
The EndpointsDatastore (de)serializes ContentApp info for persistent storage. The ContentApp SupportedCluster set is not included in the serialization.
[Solution]
Extend EndpointsDatastore serializeContentApp/deserializeContenApp methods to include the SupportedCluster set.
[Test]
Tested using TV Android App. Here's what's been stored after serialisation