Workflow cmds

[manojdbos]

dbos workflow list
dbos workflow get
dbos workflow cancel

[manojdbos]

FYI:

I have used WorkflowStatusInternal class and _sys_db method to get workflow status. There is also a WorkflowStatus exposed by the dbos class. If you prefer that we use that, let me know

[qianl15]

Found a minor bug in _sys_db.py: "SAWarning: UPDATE statement has a cartesian product between FROM element(s) "dbos.workflow_status" and FROM element "dbos.workflow_inputs". Apply join condition(s) between each element to resolve."

It's in set_workflow_status function.

[manojdbos]

Found a minor bug in _sys_db.py: "SAWarning: UPDATE statement has a cartesian product between FROM element(s) "dbos.workflow_status" and FROM element "dbos.workflow_inputs". Apply join condition(s) between each element to resolve."

It's in set_workflow_status function.

Fixed

[qianl15]

Looks good!

[kraftp]

Shouldn't we use a standard string representation, not jsonpickle, for readability?

[manojdbos]

Do you mean print it out field by field with our own formatting ?
This is a list of workflowInformation. Python does not know how to serialize it
Even the field by field might not work as there are some fields like input that python might not be able to serialize

[kraftp]

Python might not be able to serialize it, but it can (almost) always stringify it to print it in a human-readable format. My worry with jsonpickle is that its outputs are often not human-readable. In your testing, how does it look it a workflow, say, returns a list? Is it printed in a human-readable format?