Visitar URL original
Stable 23.8 Backport of #131154 - Reject out-of-range `Decimal` scale when reading a `Field` by mkmkme · Pull Request #2867 · Altinity/ClickHouse · GitHub
Skip to content

Stable 23.8 Backport of #131154 - Reject out-of-range Decimal scale when reading a Field - #2867

Open
mkmkme wants to merge 1 commit into
releases/23.8.16from
backports/stable-23.8/131154
Open

mkmkme wants to merge 1 commit into
releases/23.8.16from
backports/stable-23.8/131154

Conversation

@mkmkme

@mkmkme mkmkme commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

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 to CHANGELOG.md):

Fixed a possible server crash when reading a Decimal value with an out-of-range scale from serialized data; such a value is now rejected with an error. (ClickHouse#131154 by @tiandiwonder)

Documentation entry for user-facing changes

...

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Unit tests
  • Performance tests
  • Aarch64 tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • CAS (content-addressed storage; Antalya only)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • OAuth (5m)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

…d-scale-bound

Reject out-of-range `Decimal` scale when reading a `Field`
@altinity-robot

altinity-robot commented Nov 13, 2026 •

Copy link
Copy Markdown
Collaborator

This is an automated comment for commit 69a0052 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
Check nameDescriptionStatus
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub✅ success
Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR⏳ pending

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants