Skip to content

Introduce jsonpickle serializers #12875

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Introduce jsonpickle serializers #12875

wants to merge 2 commits into from

Conversation

giograno
Copy link
Member

Motivation

🛝

Changes

@giograno giograno added this to the Playground milestone Jul 18, 2025
@giograno giograno added area: persistence Retain state between LocalStack runs semver: patch Non-breaking changes which can be included in patch releases labels Jul 18, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 18, 2025 •
edited
Loading

S3 Image Test Results (AMD64 / ARM64)

    2 files    2 suites   8m 51s ⏱️
  515 tests 465 ✅  50 💤 0 ❌
1 030 runs  930 ✅ 100 💤 0 ❌

Results for commit 25db45d.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 18, 2025 •
edited
Loading

Test Results - Preflight, Unit

21 863 tests  +1   20 206 ✅ +1   6m 40s ⏱️ -13s
     1 suites ±0    1 657 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 25db45d. ± Comparison against base commit 9bd5851.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both. 
tests.unit.state.test_pickle ‑ test_pickle_priority_queue

tests.unit.state.test_pickle ‑ test_pickle_priority_queue[dill]
tests.unit.state.test_pickle ‑ test_pickle_priority_queue[jsonpickle]

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 18, 2025 •
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 20s ⏱️ +10s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 25db45d. ± Comparison against base commit 9bd5851.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 18, 2025 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 20m 33s ⏱️ -9s
5 288 tests ±0  4 357 ✅ ±0  931 💤 ±0  0 ❌ ±0 
5 294 runs  ±0  4 357 ✅ ±0  937 💤 ±0  0 ❌ ±0 

Results for commit 25db45d. ± Comparison against base commit 9bd5851.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 18, 2025 •
edited
Loading

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 42m 43s ⏱️ - 1m 0s
4 929 tests ±0  4 151 ✅ ±0  778 💤 ±0  0 ❌ ±0 
4 931 runs  ±0  4 151 ✅ ±0  780 💤 ±0  0 ❌ ±0 

Results for commit 25db45d. ± Comparison against base commit 9bd5851.

♻️ This comment has been updated with latest results.

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

@dominikschubert dominikschubert Awaiting requested review from dominikschubert dominikschubert will be requested when the pull request is marked ready for review dominikschubert is a code owner

@thrau thrau Awaiting requested review from thrau thrau will be requested when the pull request is marked ready for review thrau is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area: persistence Retain state between LocalStack runs semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Milestone
Playground
Development

Successfully merging this pull request may close these issues.
1 participant
@giograno