issues
search
linkedin
/
venice
Venice, Derived Data Platform for Planet-Scale Workloads.
https://venicedb.org
BSD 2-Clause "Simplified" License
487
stars
84
forks
source link
[controller] Refactor StoresRoutes::getAllStores API
#1246
Open
lucy2329
opened
2 weeks ago
lucy2329
commented
2 weeks ago
[controller] Refactor StoresRoutes::getAllStores & StoresRoutes::getAllStoresStatuses API
Modify the
getAllStores
&
getAllStoresStatuses
API to support future gRPC migration.
Refactoring the APIs to ensure it can be called from both Spark-based HTTP server and gRPC server.
How was this PR tested?
Added Unit tests for
getAllStoresStatuses
and
ListStoresRequestTest.java
WIP for getAllStores
Does this PR introduce any user-facing changes?
[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.
[controller] Refactor StoresRoutes::getAllStores & StoresRoutes::getAllStoresStatuses API
getAllStores
&getAllStoresStatuses
API to support future gRPC migration.How was this PR tested?
getAllStoresStatuses
andListStoresRequestTest.java
Does this PR introduce any user-facing changes?