github-actions[bot] commented on code in PR #67189:
URL: https://github.com/apache/doris/pull/67189#discussion_r3868729946
##########
regression-test/suites/datatype_p0/nested_types/query/array_functions/test_array_zip_array_enumerate_uniq.groovy:
##########
@@ -17,6 +17,26 @@
suite("test_array_zip_array_enumerate_uniq", "p0") {
// ========== array-zip ==========
+ sql "SET debug_skip_fold_constant = false"
+ order_qt_array_zip_folded_null """
+ SELECT array_zip([1.1, 2.2, 3.3], [1, 2, 3], NULL)
+ """
+
+ sql "SET debug_skip_fold_constant = true"
+ order_qt_array_zip_null_first """
+ SELECT array_zip(NULL, [1, 2, 3], ['a', 'b', 'c'])
+ """
+ order_qt_array_zip_null_middle """
+ SELECT array_zip([1.1, 2.2, 3.3], NULL, [1, 2, 3])
+ """
+ order_qt_array_zip_null_last """
Review Comment:
[P2] Prevent the folded query from satisfying this backend test
This SQL is identical to `array_zip_folded_null` above. With the default SQL
cache enabled, the folded one-row result is cached; changing
`debug_skip_fold_constant` neither disables that cache nor changes its
affect-result digest. The second query can therefore be served before planning
or BE preparation, so this `null_last` case may pass without exercising the
crash path it is meant to cover. Please disable SQL cache around the
folded/unfolded sequence (the framework's `testFoldConst` helper does this), or
use a cache-distinct query-scoped `SET_VAR`, and restore the prior session
values afterward.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
