Code review comment for lp:~pbeaman/akiban-persistit/fix_1017957_stress_tests_corruption

Revision history for this message
Nathan Williams (nwilliams) wrote :

The fix looks reasonable and seems to match the explanation.

I only skimmed the Buffer toString or IntegrityCheck changes but they look fine.

Please change the first sentence of the merge prop to something descriptive (it is used for the commit message) and feel free to Approve.

review: Approve

« Back to merge proposal