amplab/tachyon — pull request #204 by aarondav (2014-05-27 21:55:52+00:00)
Title: When the master client fails to connect, it should stop trying

With the current implementation, users will end up continuously retrying to connect to the master, because of the while loop in each MasterClient method like

```
while (!mIsShutdown) {
connect();
...
}
```

Instead of returning a boolean, we will now interrupt the control flow during exceptional conditions, as this will should allow current users of this API to handle it (rather than having to change every usage of `connect()` to include a check that throws a less-helpful error message).

Note that this does require a breaking API change, as we have added checked exceptions to several public APIs. Since these APIs actually do incur IO, it seems reasonable that they might throw IOExceptions, however. The alternative would be to wrap our IOExceptions in RuntimeExceptions, not exactly the "Java Way", but at least would preserve API compatibility.