Thanks for the review Mike.
* DEFAULT_CONNECT_TIMEOUT_MILLIS - maybe make it 5 seconds default? 60 seconds seems like a long time. I know this was there from before.
* DEFAULT_REQUEST_TIMEOUT_MILLIS - maybe make it 20 seconds default? (Same as above)
2. It seems like much of the code in these classes is just parsing properties file configuration. Would be nice if it was factored out somehow (using Context object internally?) - maybe this should be another JIRA
The rational behind having properties based factory methods was to eventually adopt a properties file based client configuration. So RpcClientFactory.getInstance(CONFIG_FILE_NAME) will return a client that is configured vi the property file CONFIG_FILE_NAME. Although it could internally construct a Context to do getInteger() etc calls, but that would be an implementation detail. I would much rather leave it as is for now.
3. Doesn't look like the commons-compress addition to pom.xml is used.