Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2443#discussion_r200892993
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/spark/KeyVal.scala
---
@@ -141,9 +141,10 @@ class RestructureResultImpl extends
RestructureResult[Int, Boolean] {
}
trait RefreshResult[K, V] extends Serializable {
- def getKey(key: String, value: Boolean): (K, V)
+ def getKey(key: String, value: (String, Boolean)): (K, V)
--- End diff --
please add comment to describe the modification
---