ported option to specify node preferences when connecting to a cluster

[fschaper]

No description provided.

[openreply-dleinhaeuser]

Ich habe noch ein Verständnisproblem, wo die index.d.ts den Bezeichner NodePreference (vs. nodePreference) herbekommt, aber das würde TS einem ja wohl vor die Füße werfen, wenn das nicht ginge.

Ansonsten fällt mir auf, dass die 1. Master-Preference nach meinem Verständnis nicht implementiert ist und B) soll die Node-Preference den Nodetypen bevorzugen, oder ausschliesslich diesen Nodetypen verwenden (was das ist, was momentan implementiert ist)?

[openreply-dleinhaeuser]

Sonst fällt mir nur auf, dass man das randomShuffle unter den switch-Block verlegen könnte, da es anscheinend immer passieren soll.

[fschaper]

Ansonsten fällt mir auf, dass die 1. Master-Preference nach meinem Verständnis nicht implementiert ist und B) soll die Node-Preference den Nodetypen bevorzugen, oder ausschliesslich diesen Nodetypen verwenden (was das ist, was momentan implementiert ist)?

ist dem .NET code entlehnt. Wenn du dir src/core/clusterDnsEndPointDiscoverer.js ab Zeile 217 anschaust, siehst du, dass die Liste der Verfügbaren Nodes nach einer Score sortiert wird. Master ist dabei immer oben. Das Default-Verhalten ist also, dass sich der Code immer mit einer Master-Node verbindet (sofern verfügbar) und nur unter der Einstellung Slave oder Random einen anderen Node-Type in Betracht zieht.

[fschaper]

Sonst fällt mir nur auf, dass man das randomShuffle unter den switch-Block verlegen könnte, da es anscheinend immer passieren soll.

s.o. - das soll nur passieren, wenn es sich nicht um die Master Präferenz handelt. Auch dies ist quasi aus dem .NET code geportet und entspricht nicht zwingend meiner Auffassung, wie man das ausdrucksstärker definieren kann. Ich glaube das ist im originalem Code dem Umstand geschuldet, dass das Node-Handling (Random/Slave) zu einem späteren Zeitpunkt hinzugenommen wurde.