Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2400#discussion_r199315264
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/CarbonLateDecodeStrategy.scala
---
@@ -673,4 +673,26 @@ private[sql] class CarbonLateDecodeStrategy extends
SparkStrategy {
supportCodegen && vectorizedReader.toBoolean &&
cols.forall(_.dataType.isInstanceOf[AtomicType])
}
+
+ private def createHadoopFSRelation(relation: LogicalRelation) = {
+ val sparkSession = relation.relation.sqlContext.sparkSession
+ relation.catalogTable match {
+ case Some(catalogTable) =>
+ HadoopFsRelation(new CatalogFileIndex(
--- End diff --
If parameter list is long, it is better to add the parameter name also for
better readability.
---
