Skip to content
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

[CT-1746] Flags.MP_CONTEXT is not JSON serializable #6509

Closed
Tracked by #6706
, #6631
...
MichelleArk opened this issue Jan 4, 2023 · 1 comment · Fixed by #6650
Closed
Tracked by #6706
, #6631
...

[CT-1746] Flags.MP_CONTEXT is not JSON serializable #6509

MichelleArk opened this issue Jan 4, 2023 · 1 comment · Fixed by #6650
Assignees
@ChenyuLInx
Labels
python_api Issues related to dbtRunner Python entry point

Comments

@MichelleArk
Copy link
Contributor

One of the many exceptions raised during integration test runs on the feature/click-cli branch is:

TypeError: Object of type SpawnContext is not JSON serializable

There are a couple options to mitigate the issue:

  1. Remove MP_CONTEXT from flags (where would be the best place to put it instead?)
  2. Avoid serializing MP_CONTEXT
@MichelleArk MichelleArk added the python_api Issues related to dbtRunner Python entry point label Jan 4, 2023
@github-actions github-actions bot changed the title Flags.MP_CONTEXT is not JSON serializable [CT-1746] Flags.MP_CONTEXT is not JSON serializable Jan 4, 2023
This was referenced Jan 4, 2023
Closed
Merged
@iknox-fa iknox-fa mentioned this issue Jan 17, 2023
Closed
28 tasks
@iknox-fa
Copy link
Contributor

@ChenyuLInx Can you make sure that this has been fixed? Per BLG we really think this doesn't belong in flags and should re-locate it-- will open a ticket for that work.

@iknox-fa iknox-fa mentioned this issue Jan 18, 2023
Open
@ChenyuLInx ChenyuLInx self-assigned this Jan 19, 2023
@ChenyuLInx ChenyuLInx mentioned this issue Jan 19, 2023
Merged
1 task
@jtcohen6 jtcohen6 linked a pull request Jan 19, 2023 that will close this issue
fix MP_CONTEXT is not JSON serializable #6650
Merged
1 task
@leahwicz leahwicz mentioned this issue Jan 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChenyuLInx ChenyuLInx

Labels
python_api Issues related to dbtRunner Python entry point
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix MP_CONTEXT is not JSON serializable dbt-labs/dbt-core
3 participants
@MichelleArk @ChenyuLInx @iknox-fa