search

hazelcast / hazelcast-nodejs-client

Hazelcast Node.js Client
https://hazelcast.com/clients/node-js/
Apache License 2.0
150 stars 59 forks source link

Change doTryNextCluster logic to make it the same. [API-1354] #1403

Closed harunalpak closed 2 years ago

harunalpak commented 2 years ago

Before changes, client try only maxTryCount. After code changes, client will try (candidateClusters.length * maxTryCount). For each cluster we will try maxTryCount.

codecov[bot] commented 2 years ago

Codecov Report

Merging #1403 (612882b) into master (62a2183) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1403   +/-   ##
=======================================
  Coverage   93.27%   93.27%           
=======================================
  Files         464      464           
  Lines       16387    16387           
  Branches     1333     1333           
=======================================
  Hits        15285    15285           
+ Misses        804      803    -1     
- Partials      298      299    +1
Impacted Files Coverage Δ
src/ClusterFailoverService.ts 98.07% <100.00%> (ø)
src/proxy/MultiMapProxy.ts 98.46% <0.00%> (-0.52%) :arrow_down:
src/protocol/ErrorFactory.ts 65.41% <0.00%> (+0.75%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more