[GitHub] [carbondata] QiangCai commented on a change in pull request #3950: [CARBONDATA-3889] Enable scalastyle check for all scala test code

[GitBox]

QiangCai commented on a change in pull request #3950:
URL: https://github.com/apache/carbondata/pull/3950#discussion_r504464628



##########
File path: 
index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/secondaryindex/TestCreateIndexTable.scala
##########
@@ -262,40 +262,46 @@ class TestCreateIndexTable extends QueryTest with 
BeforeAndAfterAll {
         "projectjoindate Timestamp, projectenddate Timestamp, attendance int, 
" +
         "utilization int,salary int) STORED AS CARBONDATA")
     sql(s"LOAD DATA LOCAL INPATH '$resourcesPath/data.csv' INTO " +
-        "TABLE carbontable OPTIONS('DELIMITER'=',', 
'BAD_RECORDS_LOGGER_ENABLE'='FALSE', 'BAD_RECORDS_ACTION'='FORCE')")
+        "TABLE carbontable OPTIONS('DELIMITER'=',', 
'BAD_RECORDS_LOGGER_ENABLE'='FALSE', " +
+        "'BAD_RECORDS_ACTION'='FORCE')")
     val withoutIndex =
-      sql("select empno from carbontable where empname = 'ayushi' or empname = 
'krithin' or empname = 'madhan'")
+      sql("select empno from carbontable " +

Review comment:
       change for all places

##########
File path: 
index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/secondaryindex/TestCreateIndexTable.scala
##########
@@ -262,40 +262,46 @@ class TestCreateIndexTable extends QueryTest with 
BeforeAndAfterAll {
         "projectjoindate Timestamp, projectenddate Timestamp, attendance int, 
" +
         "utilization int,salary int) STORED AS CARBONDATA")
     sql(s"LOAD DATA LOCAL INPATH '$resourcesPath/data.csv' INTO " +
-        "TABLE carbontable OPTIONS('DELIMITER'=',', 
'BAD_RECORDS_LOGGER_ENABLE'='FALSE', 'BAD_RECORDS_ACTION'='FORCE')")
+        "TABLE carbontable OPTIONS('DELIMITER'=',', 
'BAD_RECORDS_LOGGER_ENABLE'='FALSE', " +
+        "'BAD_RECORDS_ACTION'='FORCE')")
     val withoutIndex =
-      sql("select empno from carbontable where empname = 'ayushi' or empname = 
'krithin' or empname = 'madhan'")
+      sql("select empno from carbontable " +
+          "where empname = 'ayushi' or empname = 'krithin' or empname = 
'madhan'")
         .collect().toSeq
-    sql("create index empnameindex on table carbontable 
(workgroupcategoryname,empname) AS 'carbondata'")
+    sql(
+      "create index empnameindex on table carbontable (" +
+      "workgroupcategoryname,empname) AS 'carbondata'")
 
-    checkAnswer(sql("select empno from carbontable where empname = 'ayushi' or 
empname = 'krithin' or empname = 'madhan'"),
+    checkAnswer(sql(

Review comment:
       change for all places




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org