HBASE-27038 CellComparator should extend Serializable (apache#4492)

Signed-off-by: Andrew Purtell <apurtell@apache.org> Change-Id: I41fb5ddd81e682490c117566dbe24c6a7dfe4732
vinayakphegde · Jun 28, 2022 · 760bc9d · 760bc9d
1 parent 1f99770
commit 760bc9d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/CellComparator.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/CellComparator.java
@@ -17,6 +17,7 @@
  */
 package org.apache.hadoop.hbase;
 
+import java.io.Serializable;
 import java.nio.ByteBuffer;
 import java.util.Comparator;
 import org.apache.hadoop.hbase.util.ByteBufferUtils;
@@ -30,7 +31,7 @@
  */
 @InterfaceAudience.Public
 @InterfaceStability.Evolving
-public interface CellComparator extends Comparator<Cell> {
+public interface CellComparator extends Comparator<Cell>, Serializable {
   /**
    * A comparator for ordering cells in user-space tables. Useful when writing cells in sorted order
    * as necessary for bulk import (i.e. via MapReduce).

diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/CellComparatorImpl.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/CellComparatorImpl.java
@@ -47,6 +47,8 @@
 @InterfaceStability.Evolving
 public class CellComparatorImpl implements CellComparator {
 
+  private static final long serialVersionUID = 8186411895799094989L;
+
   /**
    * Comparator for plain key/values; i.e. non-catalog table key/values. Works on Key portion of
    * KeyValue only.