Skip to content

fix: Deserialization issue#481

Merged
Prajwal-Microsoft merged 1 commit intopsl-cps-v2from
psl-docker-build-and-push-v2
Mar 27, 2026
Merged

fix: Deserialization issue#481
Prajwal-Microsoft merged 1 commit intopsl-cps-v2from
psl-docker-build-and-push-v2

Conversation

@Prajwal-Microsoft
Copy link
Copy Markdown
Collaborator

Purpose

This pull request improves the serialization of processed output in the GapExecutor to handle Python datetime objects, preventing serialization errors during workflow execution. It also adds comprehensive unit tests to ensure datetime values are correctly serialized.

Serialization improvements:

  • Added a _json_default method to GapExecutor to convert datetime, date, and time objects to ISO format strings for JSON serialization, and updated the serialization logic in _serialize_processed_output to use this method. This prevents failures when serializing processed output containing these types. [1] [2] [3]

Testing enhancements:

  • Introduced a new TestSerializeProcessedOutput test class to verify that GapExecutor._serialize_processed_output correctly serializes dictionaries containing nested datetime values. [1] [2]

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@Prajwal-Microsoft Prajwal-Microsoft merged commit 2d2200d into psl-cps-v2 Mar 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant