1
Vote

WorkerAddress.GetHashCode has wrong semantics

description

Two items for which Equals would return true should have the same value for GetHashCode (otherwise things like Dictionary key lookup will fail).
Currently the default implementation of GetHashCode is used, which does not match the implementation of Equals.
This would be a better implementation:
    public override int GetHashCode()
    {
return (_nodeId << 16) | _workerId;
    }

comments