Index: hbase-client/src/main/java/org/apache/hadoop/hbase/coprocessor/ColumnInterpreter.java =================================================================== --- hbase-client/src/main/java/org/apache/hadoop/hbase/coprocessor/ColumnInterpreter.java (revision 1581106) +++ hbase-client/src/main/java/org/apache/hadoop/hbase/coprocessor/ColumnInterpreter.java (working copy) @@ -60,7 +60,6 @@ Q extends Message, R extends Message> { /** - * TODO: when removing {@link #getValue(byte[], byte[], KeyValue)}, this method should be made abstract * * @param colFamily * @param colQualifier @@ -68,26 +67,10 @@ * @return value of type T * @throws IOException */ - public T getValue(byte[] colFamily, byte[] colQualifier, Cell c) - throws IOException { - // call the deprecated method for compatiblity. - KeyValue kv = KeyValueUtil.ensureKeyValue(c); - return getValue(colFamily, colQualifier, kv); - } + public abstract T getValue(byte[] colFamily, byte[] colQualifier, Cell c) + throws IOException; /** - * This method used to be abstract, and is preserved for compatibility and easy of conversion - * from 0.94->0.96. - * - * Please override {@link #getValue(byte[], byte[], Cell)} instead. - */ - @Deprecated - public T getValue(byte[] colFamily, byte[] colQualifier, KeyValue kv) - throws IOException { - return null; - } - - /** * @param l1 * @param l2 * @return sum or non null value among (if either of them is null); otherwise