bookkeeper icon indicating copy to clipboard operation
bookkeeper copied to clipboard

Improve only one region need localNode in regionAwarePolicy

Open TakaHiR07 opened this issue 2 years ago • 0 comments

Motivation

Now in regionAwarePolicy, each region has the same localNode. Such as:

  • region1 - localNode is /region1/rack1
  • region2 - localNode is /region1/rack1
  • region3 - localNode is /region1/rack1

This is not correct and would generate more confusing warn log in newEnsemble() 企业微信截图_f12217d6-af21-41d6-b2de-c1c484a0d788

This pr is a supplement of https://github.com/apache/bookkeeper/pull/4091. We should fix the localNode not update first.

Changes

  1. only one region need localNode. So reset the localNode in other region
  2. add test

TakaHiR07 avatar Sep 26 '23 09:09 TakaHiR07