Description
I don't like the names of the filter methods in RowFilterInterface. They don't really tell how the methods are being used in the implementation of scanners.
I'd like to change:
- filter(Text) to filterRow(...)
- filter(Text, Text, byte[]) to filterColumn(...)
and the worst one is - filterNotNull(SortedMap<Text, byte[]>). This should be filterRow(Text, SortedMap<Text, byte[]>) (so we add the row key/).
It may be nice to have timestamps in the methods as well?
Also the java doc could be cleaned and improved to tell how the filtering is implemented (check rows keys first, then check each individual columns, finally check the assembled row)
Upon positive feedback, and I'll create a patch.