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

    https://github.com/apache/carbondata/pull/1296#discussion_r137919323
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/iud/UpdateCarbonTableTestCase.scala
 ---
    @@ -434,6 +434,43 @@ class UpdateCarbonTableTestCase extends QueryTest with 
BeforeAndAfterAll {
         sql("DROP TABLE IF EXISTS default.carbon1")
       }
     
    +  test("update table in carbondata with rand() ") {
    +    sql(
    +      s"""CREATE TABLE if not exists rand (CUST_ID int,CUST_NAME 
String,ACTIVE_EMUI_VERSION string,
    +         | DOB timestamp, DOJ timestamp, BIGINT_COLUMN1 
bigint,BIGINT_COLUMN2 bigint,
    +         | DECIMAL_COLUMN1 decimal(30,10), DECIMAL_COLUMN2 
decimal(36,10),Double_COLUMN1 double,
    +         | Double_COLUMN2 double,INTEGER_COLUMN1 int) STORED BY 
'org.apache.carbondata.format'
    +         | 
TBLPROPERTIES('DICTIONARY_INCLUDE'='CUST_ID,CUST_NAME,ACTIVE_EMUI_VERSION,DOB,DOJ,BIGINT_COLUMN1,
    +         | 
BIGINT_COLUMN2,DECIMAL_COLUMN1,DECIMAL_COLUMN2,Double_COLUMN1,Double_COLUMN2,INTEGER_COLUMN1')"""
    +        .stripMargin).collect()
    +
    +    sql(s"""LOAD DATA INPATH '$resourcesPath/IUD/2000_UniqData.csv' into 
table rand OPTIONS('DELIMITER'=',' , 
'QUOTECHAR'='"','BAD_RECORDS_ACTION'='FORCE','FILEHEADER'='CUST_ID,CUST_NAME,ACTIVE_EMUI_VERSION,DOB,DOJ,BIGINT_COLUMN1,BIGINT_COLUMN2,DECIMAL_COLUMN1,DECIMAL_COLUMN2,Double_COLUMN1,Double_COLUMN2,INTEGER_COLUMN1')""").collect
    --- End diff --
    
    Try to use the csv files which are already existed for this testcase. I 
don't see any reason to add new csv for this simple testcase.


---

Reply via email to