client: remove the write operation on *registry.Node in LoadNodeConfig to avoid data race during selecting Node

[liuzengh]

cherry-pick from internal code within the company

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cf5ca25) 85.80909% compared to head (a7a6ef1) 85.72580%.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main        #138         +/-   ##
===================================================
- Coverage   85.80909%   85.72580%   -0.08329%     
===================================================
  Files            189         189                 
  Lines          16123       16127          +4     
===================================================
- Hits           13835       13825         -10     
- Misses          1736        1740          +4     
- Partials         552         562         +10     

Flag	Coverage Δ
unittests	85.72580% <100.00000%> (-0.08329%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
client/client.go	86.62791% <100.00000%> (ø)
client/options.go	88.38527% <ø> (-0.06543%)	⬇️
client/stream.go	77.90698% <100.00000%> (+1.65698%)	⬆️

... and 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.