Re: Review Request: Review compaction heuristic and move compaction code out so standalone and independently testable

2010-10-25 Thread Jean-Daniel Cryans
--- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1078/#review1655 ---

Re: Review Request: Review compaction heuristic and move compaction code out so standalone and independently testable

2010-10-23 Thread Jonathan Gray
--- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1078/ --- (Updated 2010-10-22 23:39:07.148296) Review request for hbase, stack, Nicolas,

Re: Review Request: Review compaction heuristic and move compaction code out so standalone and independently testable

2010-10-23 Thread stack
--- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1078/#review1634 --- This is great. Don't we already have a standalone compaction issues?

Re: Review Request: Review compaction heuristic and move compaction code out so standalone and independently testable

2010-10-23 Thread Jonathan Gray
On 2010-10-22 23:57:25, stack wrote: This is great. Don't we already have a standalone compaction issues? Should kill it w/ this issue? Some broad questions below. Do you find your new selector better than old? Have not run the numbers against old one yet. That's next. On

Re: Review Request: Review compaction heuristic and move compaction code out so standalone and independently testable

2010-10-23 Thread Jonathan Gray
--- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1078/ --- (Updated 2010-10-23 00:39:51.093557) Review request for hbase, stack, Nicolas,

Re: Review Request: Review compaction heuristic and move compaction code out so standalone and independently testable

2010-10-23 Thread Jonathan Gray
--- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1078/ --- (Updated 2010-10-23 00:48:28.858137) Review request for hbase, stack, Nicolas,