search

magicknot / PADI-DSTM

A simplified distributed software transactional system.
2 stars 0 forks source link

PadIntNotFoundException shouldn't assume that the PadInt is in a given server #9

Closed magicknot closed 10 years ago

magicknot commented 10 years ago

PadIntNotFoundException receives two arguments (the UID and the ServerID) but the ServerID shouldn't be always required.

(e.g. if the PadInt doesn't exist at all, it won't be found in any server, so the exception should be thrown without that information)

castalheiro commented 10 years ago

That exception is thrown when you try to access a PadInt providing the server id, that's why. If you know about any other case where PadInt is not found and it would be interesting to throw the same exception, tell me about it.

2014-04-09 0:54 GMT+01:00 David Duarte notifications@github.com:

PadIntNotFoundException receives two arguments (the UID and the ServerID) but the ServerID shouldn't be always required.

(e.g. if the PadInt doesn't exist at all, it won't be found in any server, so the exception should be thrown without that information)

Reply to this email directly or view it on GitHubhttps://github.com/magicknot/PADI-DSTM/issues/9 .

José Cavalheiro

magicknot commented 10 years ago

Using the Load Balancers proposed on #8 the client may try to read a PadInt that doesn't exist, and so the respective LB will fail to find it and should throw this exception just saying that.

castalheiro commented 10 years ago

That's Master server responsability. Loadbalancer should only be concerned about what's the best server to store a new PadInt, without knowing which one. If you try to take some of master's responsability you will find alot of issues which are not related with loadBalancer at all, beware!

2014-04-09 11:19 GMT+01:00 David Duarte notifications@github.com:

Using the Load Balancers proposed on #8https://github.com/magicknot/PADI-DSTM/issues/8the client may try to read a PadInt that doesn't exist, and so the respective LB will fail to find it and should throw this exception just saying that.

Reply to this email directly or view it on GitHubhttps://github.com/magicknot/PADI-DSTM/issues/9#issuecomment-39948649 .

José Cavalheiro