[HBASE-13286] Minimum timeout for a rpc call could be 1 ms instead of 2 seconds

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.0.0, 0.98.12
• Fix Version/s: 2.0.0
• Component/s: Client
• Hadoop Flags: Reviewed
• Assignee: Nicolas Liochon
• Reporter: Nicolas Liochon
• Created: 19/Mar/15 14:27
• Updated: 23/Jun/22 20:45
• Resolved: 20/Mar/15 19:47

Description

There is a check in the client to be sure that we don't use a timeout of zero (i.e. infinite). This includes setting the minimal time out for a rpc timeout to 2 seconds. However, it makes sense for some calls (typically gets going to the cache) to have much lower timeouts. So it's better to do the check vs. zero but with a minimal timeout of 1.

I fixed a typo & a wrong comment in this patch as well. I don't understand this code:

      // t could be a RemoteException so go around again.
      translateException(t); // We don't use the result?

but may be it's good.