Antonio Lain

Results 10 comments of Antonio Lain

>Agreed. I would call this try_update_options and only swap if the connection succeeds. Also I find leaving the old options to be scary since now things aren't in sync. This...

I'm starting with Go this sprint so I need an issue. There is an epic ticket but, yes, we will need to create tickets for the other sdks and link...

@stephanos I think the cli-rewrite branch is dead, main becoming the new target branch for the last two weeks, you may want to merge in main instead... @cretz Can you...

This will be resolved when PR https://github.com/temporalio/documentation/pull/3107/files merges

Currently we are duplicating the Build ID and adding it to both WFTStarted and WFTCompleted for backwards compatibility, is that correct?

Hi Nikhil, I was about to start on this, but if you are interested, give it a try, and I'll help with the reviews instead...

I'm a bit confused about what is the ultimate goal and why this is needed. If we want to eliminate all side channels this is not going to work, e.g.,...

@dandavison Thanks a lot for a careful review of the docstrings, it made a huge difference!

> @antlai-temporal 👋 what's standing in the way of merging? Asking since this is the last PR before we can publish the documentation. Sorry @stephanos just saw your comment. I...

> I will refactor the `WorkflowIdConflictPolicy` enum in #1534. Thanks @mjameswh !