Skip to content

Commit

Permalink
Fix cratedb-wtf record about too large values of ulimit_hard
Browse files Browse the repository at this point in the history
Store them as STRING instead of BIGINT.
  • Loading branch information
amotl committed Dec 16, 2024
1 parent 11bc797 commit f041e80
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
## Unreleased
- Dependencies: Minimize dependencies of core installation,
defer `polars` to `cratedb-toolkit[io]`.
- Fixed `cratedb-wtf record` about too large values of `ulimit_hard`

## 2024/10/13 v0.0.29
- MongoDB: Added Zyp transformations to the CDC subsystem,
Expand Down
3 changes: 3 additions & 0 deletions cratedb_toolkit/wtf/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,9 @@ def system(self):
data["eco"] = boltons.ecoutils.get_profile(scrub=self.scrub)
# `version_info` is a list of mixed data types: [3, 11, 6, "final", 0].
data["eco"]["python"]["version_info"] = str(data["eco"]["python"]["version_info"])
# SQLParseException[Value 9223372036854775807 exceeds allowed range for column of type bigint]
# "ulimit_hard": 9223372036854775807,
data["eco"]["ulimit_hard"] = str(data["eco"]["ulimit_hard"])

Check warning on line 56 in cratedb_toolkit/wtf/core.py

View check run for this annotation

Codecov / codecov/patch

cratedb_toolkit/wtf/core.py#L56

Added line #L56 was not covered by tests
# data["libraries"] = PlatformInfo.libraries() # noqa: ERA001
return data

Expand Down

0 comments on commit f041e80

Please sign in to comment.