Do not apply lazy materialization for plans with array join

[vdimir]

Changelog category (leave one):

• Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

• Skip applying lazy materialization for plans with arrayJoin, which could lead to limit not being respected. Close Bad arrayJoin result with optimize_read_in_order and query_plan_execute_functions_after_sorting disabled #101608

[clickhouse-gh]

Workflow [PR], commit [bcda4f2]

Summary: ❌

job_name	test_name	status	info	comment
Stateless tests (amd_tsan, flaky check)		failure
	03172_error_log_table_not_empty	FAIL	cidb
	03172_error_log_table_not_empty	FAIL	cidb
	03172_error_log_table_not_empty	FAIL	cidb
Stateless tests (amd_msan, flaky check)		failure
	03172_error_log_table_not_empty	FAIL	cidb
Integration tests (amd_tsan, 2/6)		failure
	test_storage_kafka/test_batch_slow_6.py::test_kafka_handling_commit_failure	FAIL	cidb

---

AI Review

Summary

This PR prevents optimizeLazyMaterialization2 from being applied when the expression/filter chain between sorting and ReadFromMergeTree contains arrayJoin, which fixes incorrect LIMIT/OFFSET behavior reported in #101608. The added guard and new stateless test are aligned with the bug scope, and I did not find correctness, safety, or performance regressions in the changed code.

ClickHouse Rules

Item	Status	Notes
Deletion logging	➖
Serialization versioning	➖
Core-area scrutiny	✅
No test removal	✅
Experimental gate	➖
No magic constants	✅
Backward compatibility	✅
SettingsChangesHistory.cpp	➖
PR metadata quality	✅
Safe rollout	✅
Compilation time	✅

Final Verdict

• Status: ✅ Approve

[vdimir]

Stateless tests (amd_tsan, flaky check) failure
03172_error_log_table_not_empty FAIL cidb
03172_error_log_table_not_empty FAIL cidb
03172_error_log_table_not_empty FAIL cidb
Stateless tests (amd_msan, flaky check) failure
03172_error_log_table_not_empty FAIL cidb

Fix #102047

[clickhouse-gh]

LLVM Coverage Report

Metric	Baseline	Current	Δ
Lines	84.00%	84.00%	+0.00%
Functions	90.90%	90.90%	+0.00%
Branches	76.50%	76.40%	-0.10%

Changed lines: 87.10% (27/31) · Uncovered code

Full report · Diff report