Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2902#discussion_r231014429 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/iud/UpdateCarbonTableTestCase.scala --- @@ -772,12 +772,33 @@ class UpdateCarbonTableTestCase extends QueryTest with BeforeAndAfterAll { sql("""drop table if exists iud.dest33_part""") } + test("check data after update with row.filter pushdown as false") { + CarbonProperties.getInstance().addProperty(CarbonCommonConstants + .CARBON_PUSH_ROW_FILTERS_FOR_VECTOR, "false") + sql("""drop table if exists iud.dest33_flat""") + sql( + """create table iud.dest33_part (c1 int,c2 string, c3 short) STORED BY 'carbondata'""" + .stripMargin) + sql( + s"""LOAD DATA LOCAL INPATH '$resourcesPath/IUD/negativevalue.csv' INTO table iud + |.dest33_part options('header'='false')""".stripMargin) + sql( + """update iud.dest33_part d set (c1) = (5) where d.c1 = 0""".stripMargin).show() + checkAnswer(sql("select c3 from iud.dest33_part"), Seq(Row(-300), Row(0), Row(-200), Row(700) + , Row(100), Row(-100), Row(null))) + sql("""drop table if exists iud.dest33_part""") + CarbonProperties.getInstance().addProperty(CarbonCommonConstants + .CARBON_PUSH_ROW_FILTERS_FOR_VECTOR, "true") --- End diff -- After test case completion we should set the default value for `CARBON_PUSH_ROW_FILTERS_FOR_VECTOR`?...default property is false so I think at the start of test case no need to modify the property value
---