pforpallav / cluster-server Goto Github PK

Hi Kumar,

This is a review of your assignement 2.

1. I have the latest zeromq installed, and alecthomas's zmq library is not up-to-date, so I have not been able to test your submission. Please use the github.com/pebbe/zmq4 library instead.
2. Creating a new socket and connecting to it (in Sender) for every single message is extremely inefficient. Please refer to my notes on Piazza about assignment 2. Also don't use REQ/REP sockets.
3. You are doing an all-to-all send / receive test, which is a good thing. But that is precisely what the broadcast Pid in the envelope is for. The client should not have to enumerate each peer in the cluster.
4. The test strategy may not work if you change the socket type (which you must). You expect the messages to be received in the same order that they were sent in. Shouldn't have that expectation.
5. Reciever is misspelt. It is Receiver. Here's the rule: I before E except C. Also "AddPear" in README