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