Clustering : ShardTransaction ReadData should be process synchronously

ShardTransaction should process ReadData the same way it processes DataExists. There is no good reason or value to making it async in the ShardTransaction actor. Also this allows all operations on a transaction to be executed in proper order (i.e) a write cannot happen before a read and so on.

Logging needs to be enhanced to log exceptions when a read fails.