kunal642 commented on a change in pull request #3899:
URL: https://github.com/apache/carbondata/pull/3899#discussion_r477173788
##########
File path:
integration/spark/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/TestLoadDataGeneral.scala
##########
@@ -310,7 +310,8 @@ class TestLoadDataGeneral extends QueryTest with
BeforeAndAfterEach {
val tableStatusFile =
CarbonTablePath.getTableStatusFilePath(carbonTable.getTablePath)
FileFactory.getCarbonFile(tableStatusFile).delete()
sql("insert into stale values('k')")
- checkAnswer(sql("select * from stale"), Row("k"))
+ // if table lose tablestatus file, the system should keep all data.
+ checkAnswer(sql("select * from stale"), Seq(Row("k"), Row("k")))
Review comment:
I think this behaviour would be harrmful when the segment is marked for
delete..If the segment is not removed then the results would be incorrect.
----------------------------------------------------------------
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
