[VL] Remaining filter behavior change after velox switch to meta velox

[zhli1142015]

Backend

VL (Velox)

Bug description

Before

After

Query:

spark.sql("""
select count(ss_item_sk), ss_addr_sk, ss_ext_sales_price, ss_sold_date_sk from store_sales where IsNull(ss_addr_sk) and IsNotNull(ss_item_sk)
group by ss_addr_sk, ss_ext_sales_price, ss_sold_date_sk
""").collect

Plan Node: 
-- Project[expressions: (n2_4:INTEGER, murmur3hash(42,"n0_1","n0_2","n0_3")), (n2_5:INTEGER, "n0_1"), (n2_6:DECIMAL(7,2), "n0_2"), (n2_7:INTEGER, "n0_3"), (n2_8:BIGINT, "n1_3")] -> n2_4:INTEGER, n2_5:INTEGER, n2_6:DECIMAL(7,2), n2_7:INTEGER, n2_8:BIGINT
  -- Aggregation[PARTIAL [n0_1, n0_2, n0_3] n1_3 := count("n0_0")] -> n0_1:INTEGER, n0_2:DECIMAL(7,2), n0_3:INTEGER, n1_3:BIGINT
    -- TableScan[table: hive_table, range filters: [(ss_item_sk, Filter(IsNotNull, deterministic, null not allowed))], remaining filter: (isnull("ss_addr_sk"))] -> n0_0:INTEGER, n0_1:INTEGER, n0_2:DECIMAL(7,2), n0_3:INTEGER

Before isnull is added in https://github.com/oap-project/velox/blob/update/velox/expression/ExprToSubfieldFilter.cpp#L509, but not now.

else if (call.name() == "is_null" || call.name() == "isnull")

Spark version

None

Spark configurations

No response

System information

No response

Relevant logs

No response

[zhli1142015]

cc @rui-mo