search

CentaurusInfra / global-resource-service

Apache License 2.0
0 stars 4 forks source link

merge branch simulator-list-watch to main #168

Closed sonyafenge closed 1 year ago

sonyafenge commented 1 year ago

tested on [Nodequery][Daily] [metrics][5M][LS][CrossRegion]scheduler100x50K, simulator5x40x25k, Logs logs copied to sonyadev4:/home/sonyali/grs/logs/5000000/Daily/091422-210046

Sindica commented 1 year ago

This change needs to be further reviewed before merging into master. Better use separate pull requests to split out individual changes.

yb01 commented 1 year ago

This change needs to be further reviewed before merging into master. Better use separate pull requests to split out individual changes.

i believe the intention of this PR is to run the query + daily patter change with the list-watch model, not meant to merge the lw branch to main. as you mentioned, it is not ready to merge yet -- at least the lw end to end support function definitions should be there for the lw model.

sonyafenge commented 1 year ago

close per this is not ready to merge