It really assumes a String, String map ... Is it just because Properties is synced?
If so, then when moving to 1.5 we should declare the Map with Map<String, String> because currently if anyone will pass anything other than Strings, the code will fail with a ClassCastException in ChecksumIndexOutput.writeStringStringMap. Shai