maheshrajus commented on code in PR #4276:
URL: https://github.com/apache/carbondata/pull/4276#discussion_r908062873
##########
integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/standardpartition/StandardPartitionTableQueryTestCase.scala:
##########
@@ -494,9 +494,8 @@ test("Creation of partition table should fail if the
colname in table schema and
sql(s"""ALTER TABLE staticpartitionlocloadother_new DROP
PARTITION(empname='ravi')""")
checkAnswer(sql(s"select count(deptname) from
staticpartitionlocloadother_new"), Seq(Row(10)))
sql(s"""alter table staticpartitionlocloadother_new add partition
(empname='ravi') location '$location'""")
- checkAnswer(sql(s"select count(deptname) from
staticpartitionlocloadother_new"), Seq(Row(20)))
sql(s"""LOAD DATA local inpath '$resourcesPath/data.csv' INTO TABLE
staticpartitionlocloadother_new partition(empname='ravi') OPTIONS('DELIMITER'=
',', 'QUOTECHAR'= '"')""")
- checkAnswer(sql(s"select count(deptname) from
staticpartitionlocloadother_new"), Seq(Row(30)))
+ checkAnswer(sql(s"select count(deptname) from
staticpartitionlocloadother_new"), Seq(Row(20)))
Review Comment:
before we are not removing partition folders so when load again the count
was 30. With new change we are removing partition folders when we performed
drop partition. as per new change modified the test case and test case is
passing.
--
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.
To unsubscribe, e-mail: dev-unsubscr...@carbondata.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
