commit | 4624bac91058c881436c2c8411c6106269640823 | [log] [tgz] |
---|---|---|
author | Rui Qiu <ruiqiu@google.com> | Tue Oct 12 16:41:18 2021 -0700 |
committer | Rui Qiu <ruiqiu@google.com> | Mon Oct 18 15:53:32 2021 -0700 |
tree | 28d4a82cfac5fc193ab49cd51ebf81c93af0531a | |
parent | 947f5f4c365ad22cfd50e3e7e8cfcc4c657bdbbd [diff] |
[RESTRICT AUTOMERGE] Check both MetricsConfig version for add and remove APIs In add/remove config API, the validity of a MetricsConfig is checked by MetricsConfigStore. If the MetricsConfigStore returns a success signal to CarTelemetryService, the add/remove operations will be applied on the rest of CarTelemetryService (e.g. ResultStore and DataBroker) This CL also refactor the internal APIs to make sure their signatures are consistent. Bug: 198792767 Test: atest CarServiceUnitTest:DataBrokerTest Test: atest CarServiceUnitTest:DataBrokerControllerTest Test: atest CarServiceUnitTest:MetricsConfigStoreTest Test: atest CarServiceUnitTest:ResultStoreTest Test: atest CarServiceUnitTest:CarTelemetryServiceTest Test: atest CarServiceUnitTest:CarTelemetryManagerTest Change-Id: Ia2a0244ef5f876e15e80ee0b5134891808ab5cf7
Native (C++) code format is required to be compatible with .clang-format file. Run
git clang-format --style=file --extension='h,cpp,cc' HEAD~
Note that clang-format is not desirable for Android java files. Therefore the command line above is limited to specific extensions.