linkedin / rest.li

Rest.li is a REST+JSON framework for building robust, scalable service architectures using dynamic discovery and simple asynchronous APIs.
rest.li
Other
2.51k stars 546 forks source link

simplify symlink subscription in xds flow #939

Closed bohhyang closed 1 year ago

bohhyang commented 1 year ago

Summary

Current Observer only has a symlink for cluster config /d2/clusters/$xxxMaster, but not a counterpart symlink for uri parent /d2/uris/$xxxMaster. In fact, since cluster symlink and uri parent symlink always point to the same actual node name, and it's a redundancy and a burden for the symlink-update tool to maintain two symlinks for the same actual node name. We optimize xds flow to use the cluster symlink to listen to the actual nodes for both cluster and uri parent. Once all apps migrate off ZK flow, we don't need to maintain the uri parent symlink anymore.

Test Done

updated unit tests.