Skip to content
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

The KVClientTable (and also other tables) is not copy/move-save. #45

Open
Yuzhen11 opened this issue Jan 15, 2018 · 0 comments
Open

The KVClientTable (and also other tables) is not copy/move-save. #45

Yuzhen11 opened this issue Jan 15, 2018 · 0 comments

Comments

@Yuzhen11
Copy link
Owner

Copy:
KVClientTable should not be copied, so the copy constructor and copy assignment operator should be disable.
Move:
Ideally, KVClientTable can be moved. But actually it is not the case. If we move a table, the recv_kvs_ in kv_table_box is also moved. However, the original recv_kvs_ was captured and registered into the callback_runner_ which will store the get requests. The moved recv_kvs_ will not receive the get requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant