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
---
