Skip to content

Cherry pick #91846 to 25.11: Allow implicit type conversion in Array to QBit cast#92302

Merged
robot-clickhouse merged 10 commits intobackport/25.11/91846from
cherrypick/25.11/91846
Dec 16, 2025
Merged

Cherry pick #91846 to 25.11: Allow implicit type conversion in Array to QBit cast#92302
robot-clickhouse merged 10 commits intobackport/25.11/91846from
cherrypick/25.11/91846

Conversation

@robot-clickhouse
Copy link
Copy Markdown
Member

Original pull-request #91846

Do not merge this PR manually

This pull-request is a first step of an automated backporting.
It contains changes similar to calling git cherry-pick locally.
If you intend to continue backporting the changes, then resolve all conflicts if any.
Otherwise, if you do not want to backport them, then just close this pull-request.

The check results does not matter at this step - you can safely ignore them.

Troubleshooting

If the conflicts were resolved in a wrong way

If this cherry-pick PR is completely screwed by a wrong conflicts resolution, and you want to recreate it:

  • delete the pr-cherrypick label from the PR
  • delete this branch from the repository

You also need to check the Original pull-request for pr-backports-created label, and delete if it's presented there

The PR source

The PR is created in the CI job

Remove explicit type casts (toFloat32, toFloat64, toBFloat16) in test queries now that Array to QBit conversion handles implicit element type conversion automatically.
(1) Remove explicit copying of from/to_nested_type before lambda to avoid 'NOLINT' as we copy in lamba capture anyway, (2) move 'prepareUnpackDictionaries(..)' and creation of to_array_type outside lambda (larger capture, but computation)
We no longer need toBFloat16/toFloat32/toFloat64 functions for QBit to operate
Allow implicit type conversion in `Array` to `QBit` cast
@robot-clickhouse robot-clickhouse added pr-cherrypick Cherry-pick of merge-commit before backporting. Do not use manually - automated use only! do not test disable testing on pull request labels Dec 16, 2025
@robot-clickhouse robot-clickhouse merged commit eb6fdfd into backport/25.11/91846 Dec 16, 2025
104 of 112 checks passed
@robot-clickhouse robot-clickhouse deleted the cherrypick/25.11/91846 branch December 16, 2025 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not test disable testing on pull request pr-cherrypick Cherry-pick of merge-commit before backporting. Do not use manually - automated use only!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants