-
Notifications
You must be signed in to change notification settings - Fork 244
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.
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
EDU-3817: Use value_set/unset to update search attributes #3300
base: main
Are you sure you want to change the base?
Conversation
Thanks for picking this up and tidying @fairlydurable :) |
@EddieWhi
So that means doing something like this is deprecated: workflow.upsert_search_attributes({"CustomKeywordField": ["new-value"]}) However these methods aren't deprecated: I wonder if pyright is incorrectly reporting this? I couldn't reproduce the deprecation warning with pyright 0.8.0 installed in PyCharm. |
For reference, I ran the test using pyright v1.1.393 and python 3.12.3 Sorry if my original message wasn't clear. Explicitly, if I copy paste the example from the existing docs i.e. :
I get the following from pyright:
Although (from memory as I haven't taken the time to test again) it does actually work when run. For completeness, I tried the deprecated dictionary form you mentioned above ( Hope that helps in some way. |
What does this PR do?
Update the docs for updating python search attributes.
Notes to reviewers
Stumbled across this issue when trying this out for the first time and deprecated method didn't cleanly validate through pyright. Assume this is the official mechanism for updating search attributes in python now?