Conversation
| client_connect_config: LambdaClientConnectConfig = field( | ||
| default_factory=LambdaClientConnectConfig | ||
| ) | ||
| worker_config: WorkerConfig = field(default_factory=WorkerConfig) |
There was a problem hiding this comment.
My main design question is about this. With this decision, we auto-reuse all the worker init parameters, which is good, but the syntax looks like:
config.worker_config["task_queue"] = "my_task_queue"
Which, the key literal does autocomplete, but it's maybe not as nice as duplicating some of the more common fields so we can have
config.task_queue = "my_task_queue"
The downside there is, of course, that they're duplicative of the same fields as keys on worker_config.
I think I prefer how it is now, since the keys still autocomplete, but would like to hear input.
There was a problem hiding this comment.
We don't have a type like WorkerConfig for client connect. We probably should - I could add it in this PR but it would mean making changes to the main part of the SDK (albeit easy ones)
Sushisource
force-pushed
the
lambdaworker
branch
from
1f438ac to
f837c8a
Compare
1 participant
This is verified working in combination with this sample: temporalio/samples-python#286
At least one open design question I want to resolve before merging.