Skip to content

Commit

Permalink
MB-45380 do not use etag in rebalancer/ddl_service_manager
Browse files Browse the repository at this point in the history
rebalancer/ddl_service_manager do not use etag for
/getLocalIndexMetadata. It is better to call the rest api explicitly
with useETag=false.

Change-Id: I340d4dd70c8914122d8abc63a1c33378618d2d06
  • Loading branch information
deepkaran committed Apr 10, 2021
1 parent 1f484e6 commit dd37ff0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion secondary/indexer/ddl_service_manager.go
Expand Up @@ -1754,7 +1754,7 @@ func newMetadataProvider(clusterAddr string, nodes map[service.NodeID]bool, sett
addr, err := cinfo.GetServiceAddress(nid, common.INDEX_HTTP_SERVICE)
if err == nil {

resp, err := getWithAuth(addr + "/getLocalIndexMetadata")
resp, err := getWithAuth(addr + "/getLocalIndexMetadata?useETag=false")
if err != nil {
continue
}
Expand Down
2 changes: 1 addition & 1 deletion secondary/indexer/rebalancer.go
Expand Up @@ -1660,7 +1660,7 @@ func getLocalStats(addr string, partitioned bool) (*c.Statistics, error) {

func getLocalMeta(addr string) (*manager.LocalIndexMetadata, error) {

url := "/getLocalIndexMetadata"
url := "/getLocalIndexMetadata?useETag=false"
resp, err := getWithAuth(addr + url)
if err != nil {
l.Errorf("Rebalancer::getLocalMeta Error getting local metadata %v %v", addr+url, err)
Expand Down

0 comments on commit dd37ff0

Please sign in to comment.