[GLUTEN-8010][VL] Don't generate native metrics if filter's remainingCondition is null

[zml1206]

What changes were proposed in this pull request?

(Fixes: #8010)

How was this patch tested?

Local test. Before PR:

After PR:

[github-actions[bot]]

https://github.com/apache/incubator-gluten/issues/8010

[github-actions[bot]]

Run Gluten Clickhouse CI on x86

[github-actions[bot]]

Run Gluten Clickhouse CI on x86

[github-actions[bot]]

Run Gluten Clickhouse CI on x86

[zhztheplayer]

Thanks!

Though it feels like a workaround since there is no real filter node in Velox query plan? Do we have other better options?

[zml1206]

Though it feels like a workaround since there is no real filter node in Velox query plan? Do we have other better options?

How about set metricsUpdater to MetricsUpdater.None when getRemainingCondition is null? In this way, there is no metrics information.

[zhztheplayer]

How about set metricsUpdater to MetricsUpdater.None when getRemainingCondition is null?

Sounds fair. Let's see if it works.

BTW I think one of the promising ways is to exclude the filter node from Spark query plan as well. Though I remember there were some issues with an relevant attempt.

[zml1206]

BTW I think one of the promising ways is to exclude the filter node from Spark query plan as well. Though I remember there were some issues with an relevant attempt.

We can't exclude the filter node, because it's output may differ from child's output.

[github-actions[bot]]

Run Gluten Clickhouse CI on x86

[github-actions[bot]]

Run Gluten Clickhouse CI on x86

[zml1206]

@zhztheplayer Can you help take a look again? Thanks.