New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix issue of actions dag split #61458
fix issue of actions dag split #61458
Conversation
This is an automated comment for commit b3e9f07 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page
Successful checks
|
It failed a fast test and it seems related (doesn't fail without the change) so it seems this required a more detailed investigation. Reverting the offending PR is hard since a) there have been multiple changes after it and b) multiple fuzzer issues (aka crashes) were closed by those changes, so simply going back to the old code isn't a good option. |
data AS (SELECT '' id LIMIT 0), | ||
r AS (SELECT'' as id, 1::UInt8 as value) | ||
SELECT | ||
now() as date, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
now() as date, | |
1::UInt32 as date, |
To use correct types. In the current release it reproduces:
Code: 352. DB::Exception: Received from clickhouse-01:49000. DB::Exception: Block structure mismatch in (columns with identical name must have identical structure) stream: different columns:
date UInt32 UInt32(size = 0)
date UInt32 Const(size = 0, UInt32(size = 1)). (AMBIGUOUS_COLUMN_NAME)
@taiyang-li We are changing the code to cover the broken test in a simpler way. Do you have some other test that might fail in this case? Note that the original patch in the PR introduced multiple columns as input which lead to them being loaded unnecessarily, which is why |
It seems still can't solve the issue described here: #60206 (comment). If the patch from #60206 is removed, it breaks the invariant described in comments of
|
Integration tests (tsan) [4/6] - #59631 |
@taiyang-li in the example from #60206 (comment) split is correct. |
…f6672f49aa0ca2dd1e5a5b2fc9b7c Cherry pick #61458 to 23.3: fix issue of actions dag split
…f6672f49aa0ca2dd1e5a5b2fc9b7c Cherry pick #61458 to 23.8: fix issue of actions dag split
…0f6672f49aa0ca2dd1e5a5b2fc9b7c Cherry pick #61458 to 23.12: fix issue of actions dag split
…f6672f49aa0ca2dd1e5a5b2fc9b7c Cherry pick #61458 to 24.1: fix issue of actions dag split
…f6672f49aa0ca2dd1e5a5b2fc9b7c Cherry pick #61458 to 24.2: fix issue of actions dag split
Add a comment after #61458
Backport #61458 to 24.2: fix issue of actions dag split
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Documentation entry for user-facing changes
Continues from #60206