-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Spanner: add support for session / pool labels #5734
Conversation
6b57b0a
to
b35edfa
Compare
Read-only, set via ctor.
Use standard mocks for session CRUD.
Read-only, set via ctor. Concrete classes plumb it through their ctors.
33f271d
to
bcc8fde
Compare
/cc @snehashah16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, except for the note I added. I'll approve once you address that.
labels = {} | ||
self._labels = labels | ||
|
||
@property |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Labels are assigned to a pool at creation time, and then get propagated to the sessions created by the pool.