TestFullLogReconstruction has duplicate config setting

The 0.98 branch's version of TestFullLogReconstruction has a duplicate config setting during setupBeforeClass

54    // faster failover with cluster.shutdown();fs.close() idiom
55    c.setInt("hbase.ipc.client.connect.max.retries", 1);
56    c.setInt("hbase.ipc.client.connect.max.retries", 1);

the 0.98.4 release has a line for setting this config and for the HDFS version. the branch-1 and master versions of this set of patches have only the setting of the hbase one, so I think that's the correct behavior.