Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2059#discussion_r174193598
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/sort/sortdata/NewRowComparatorForNormalDims.java
---
@@ -16,13 +16,16 @@
*/
package org.apache.carbondata.processing.sort.sortdata;
+import java.io.Serializable;
import java.util.Comparator;
/**
* This class is used as comparator for comparing dims which are non high
cardinality dims.
* Here the dims will be in form of int[] (surrogates) so directly
comparing the integers.
*/
-public class NewRowComparatorForNormalDims implements Comparator<Object[]>
{
+public class NewRowComparatorForNormalDims implements
Comparator<Object[]>, Serializable {
+ private static final long serialVersionUID = 201803131547L;
--- End diff --
please change the serialVersionUID
---