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.
---