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

    https://github.com/apache/carbondata/pull/2917#discussion_r242500505
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/iud/TestShowIUDRowCount.scala
 ---
    @@ -0,0 +1,60 @@
    +package org.apache.carbondata.spark.testsuite.iud
    +
    +import org.apache.spark.sql.Row
    +import org.apache.spark.sql.test.util.QueryTest
    +import org.scalatest.{BeforeAndAfterAll, BeforeAndAfterEach}
    +
    +class TestShowIUDRowCount extends QueryTest with BeforeAndAfterEach with 
BeforeAndAfterAll {
    +
    +  override protected def beforeAll(): Unit = {
    +    dropTable("iud_rows")
    +  }
    +
    +  override protected def beforeEach(): Unit = {
    +    dropTable("iud_rows")
    +  }
    +
    +  override protected def afterEach(): Unit = {
    +    dropTable("iud_rows")
    +  }
    +
    +  test("Test show load row count") {
    +    sql("""create table iud_rows (c1 string,c2 int,c3 string,c5 string)
    +        |STORED BY 'org.apache.carbondata.format'""".stripMargin)
    +    checkAnswer(
    +      sql(s"""LOAD DATA LOCAL INPATH '$resourcesPath/IUD/dest.csv' INTO 
table iud_rows"""),
    --- End diff --
    
    I found that this test will pass if I add OPTIONS(), without/with detail 
config, in load command. And the logical plan will change from 
`LoadDataCommand`( class in spark) to `CarbonLoadDataCommand`(class in carbon). 
    If we want to show proceeded row count, we need to change the value 
`output` in these command/query plan, but we cannot do that for 
`LoadDataCommand` in spark.



---

Reply via email to