Github user keith-turner commented on the pull request:
https://github.com/apache/accumulo/pull/100#issuecomment-221384261
@joshelser I ran some performance test and did not really see a
difference. Below I compacted the file I have been testing with 3 times using
the changes in this PR.
```
2016-05-24 15:33:30,598 [tablet.Compactor] DEBUG: Compaction 2<< 24,299,468
read | 24,299,468 written | 230,337 entries/sec | 105.495 secs | 688,581,526
bytes | 6527148.453 byte/sec
2016-05-24 15:35:17,940 [tablet.Compactor] DEBUG: Compaction 2<< 24,299,468
read | 24,299,468 written | 234,137 entries/sec | 103.783 secs | 688,581,526
bytes | 6634820.019 byte/sec
2016-05-24 15:37:09,738 [tablet.Compactor] DEBUG: Compaction 2<< 24,299,468
read | 24,299,468 written | 231,960 entries/sec | 104.757 secs | 688,581,526
bytes | 6573131.399 byte/sec
```
Below I compacted the same file 3 times using 73b4fbb
```
2016-05-24 15:46:19,371 [tablet.Compactor] DEBUG: Compaction 2<< 24,299,468
read | 24,299,468 written | 233,054 entries/sec | 104.265 secs | 693,539,660
bytes | 6651701.530 byte/sec
2016-05-24 15:48:07,653 [tablet.Compactor] DEBUG: Compaction 2<< 24,299,468
read | 24,299,468 written | 235,752 entries/sec | 103.072 secs | 693,539,660
bytes | 6728691.206 byte/sec
2016-05-24 15:49:59,176 [tablet.Compactor] DEBUG: Compaction 2<< 24,299,468
read | 24,299,468 written | 233,536 entries/sec | 104.050 secs | 693,539,660
bytes | 6665446.036 byte/sec
```
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
