Skip to content

Forbid writing lower and upper bounds for complex types #85332

Merged
scanhex12 merged 4 commits intoClickHouse:masterfrom
scanhex12:fix_lower_bounds_ice_table
Aug 12, 2025
Merged

Forbid writing lower and upper bounds for complex types #85332
scanhex12 merged 4 commits intoClickHouse:masterfrom
scanhex12:fix_lower_bounds_ice_table

Conversation

@scanhex12
Copy link
Copy Markdown
Member

@scanhex12 scanhex12 commented Aug 9, 2025

Writing lower and upper bounds are not supported for complex types

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):

Writing lower and upper bounds are not supported for complex types

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Aug 9, 2025

Workflow [PR], commit [aa77f8f]

Summary:

job_name test_name status info comment
Stateless tests (amd_debug, distributed plan, s3 storage, parallel) failure
03209_parameterized_view_with_non_literal_params FAIL
Integration tests (asan, old analyzer, 3/6) failure
test_throttling/test.py::test_local_read_throttling_reload FAIL
Integration tests (aarch64, distributed plan, 4/4) failure
test_tcp_handler_interserver_listen_host/test_case.py::test_request_to_node_without_interserver_listen_host FAIL
test_tcp_handler_interserver_listen_host/test_case.py::test_request_to_node_with_interserver_listen_host FAIL
Stress test (amd_msan) failure
Server died FAIL
Hung check failed, possible deadlock found (see hung_check.log) FAIL
Killed by signal (in clickhouse-server.log) FAIL
Fatal message in clickhouse-server.log (see fatal_messages.txt) FAIL
Killed by signal (output files) FAIL
Found signal in gdb.log FAIL

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Aug 9, 2025
Copy link
Copy Markdown
Member

@antonio2368 antonio2368 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be bugfix no?

@antonio2368 antonio2368 self-assigned this Aug 11, 2025
@scanhex12
Copy link
Copy Markdown
Member Author

let's keep the label as is

@scanhex12 scanhex12 requested a review from antonio2368 August 11, 2025 08:29
@antonio2368
Copy link
Copy Markdown
Member

let's keep the label as is

Why if it fixes a bug?

@scanhex12 scanhex12 force-pushed the fix_lower_bounds_ice_table branch from b2a249d to 50fa2f8 Compare August 11, 2025 20:26
@clickhouse-gh clickhouse-gh bot added pr-bugfix Pull request with bugfix, not backported by default and removed pr-not-for-changelog This PR should not be mentioned in the changelog labels Aug 11, 2025
@scanhex12 scanhex12 added this pull request to the merge queue Aug 12, 2025
Merged via the queue into ClickHouse:master with commit c755fca Aug 12, 2025
236 of 242 checks passed
@scanhex12 scanhex12 deleted the fix_lower_bounds_ice_table branch August 12, 2025 07:43
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants