Committed to trunk. If I commit this to 0.98 one thing is as there is no Cell related changes in 0.98, so passing kv to the DeleteTracker would make it do Kv.getqualifierOffset() , kv.getQualifierLength(), kv.getTimeStamp(), kv.getTypeByte() once again which is already extracted out in the SQM itself.Andrew Purtell
But if we do a visibility deletion strategy based on DeleteTracker, then this change is very much and later would be easier in 0.99. Also would help in extracting the tag details easily from the cell itself. Would wait before committing to 0.98.

ramkrishna.s.vasudevan
added a comment - 25/Apr/14 10:01 Committed to trunk. If I commit this to 0.98 one thing is as there is no Cell related changes in 0.98, so passing kv to the DeleteTracker would make it do Kv.getqualifierOffset() , kv.getQualifierLength(), kv.getTimeStamp(), kv.getTypeByte() once again which is already extracted out in the SQM itself.
Andrew Purtell
But if we do a visibility deletion strategy based on DeleteTracker, then this change is very much and later would be easier in 0.99. Also would help in extracting the tag details easily from the cell itself. Would wait before committing to 0.98.

If I commit this to 0.98 one thing is as there is no Cell related changes in 0.98, so passing kv to the DeleteTracker would make it do Kv.getqualifierOffset() , kv.getQualifierLength(), kv.getTimeStamp(), kv.getTypeByte() once again which is already extracted out in the SQM itself.

Yes, well we need this in 0.98 so please consider making the change and attach what you commit here. +1 if what you describe above is the only change necessary.

Andrew Purtell
added a comment - 25/Apr/14 20:54 If I commit this to 0.98 one thing is as there is no Cell related changes in 0.98, so passing kv to the DeleteTracker would make it do Kv.getqualifierOffset() , kv.getQualifierLength(), kv.getTimeStamp(), kv.getTypeByte() once again which is already extracted out in the SQM itself.
Yes, well we need this in 0.98 so please consider making the change and attach what you commit here. +1 if what you describe above is the only change necessary.