[GitHub] carbondata pull request #2374: [CARBONDATA-2613] Support csv based carbon ta...

[jackylk]

Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2374#discussion_r196510839
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/carbondata/spark/rdd/CarbonScanRDD.scala
 ---
    @@ -426,6 +439,22 @@ class CarbonScanRDD[T: ClassTag](
                 
CarbonTimeStatisticsFactory.createExecutorRecorder(model.getQueryId))
               streamReader.setQueryModel(model)
               streamReader
    +        case FileFormat.EXTERNAL =>
    +          assert(storageFormat.equals("csv"),
    --- End diff --
    
    should use if check instead of assert


---