From f30de7377d1b766757e92dfcda765c9f2c25f67e Mon Sep 17 00:00:00 2001
From: Zhihong Yu <tedyu@apache.org>
Date: Tue, 25 Mar 2014 03:45:55 +0000
Subject: [PATCH] HBASE-10821 Make ColumnInterpreter#getValue() abstract

git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1581217 13f79535-47bb-0310-9956-ffa450edef68
---
 .../hbase/coprocessor/ColumnInterpreter.java  | 21 ++-----------------
 1 file changed, 2 insertions(+), 19 deletions(-)

diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/coprocessor/ColumnInterpreter.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/coprocessor/ColumnInterpreter.java
index 85c097ed45c..8ac86be4ad8 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/coprocessor/ColumnInterpreter.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/coprocessor/ColumnInterpreter.java
@@ -60,7 +60,6 @@ public abstract class ColumnInterpreter<T, S, P extends Message,
 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,24 +67,8 @@ Q extends Message, R extends Message> {
    * @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);
-  }
-
-  /**
-   * 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;
-  }
+  public abstract T getValue(byte[] colFamily, byte[] colQualifier, Cell c)
+      throws IOException;
 
   /**
    * @param l1