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

introduce ExchangeData #164

wants to merge 9 commits into
base: master


None yet
1 participant
Copy link

frankmcsherry commented Apr 2, 2019

This PR parallels timely dataflow's Data and ExchangeData traits, with the latter adding the ability to move between workers to the former.

This change allows us to use types like Rc<T> in collections, and perhaps eventually within arrangements (we will need specialized arrange methods that do not need to exchange their records).

The expected fall-out from this PR is that folks who wrote libraries that used the Data trait will likely now need to modify this to be ExchangeData. We could have left Data as it was and introduced LocalData, but then we wouldn't parallel what timely does and would likely confuse people.

cc @comnik @ryzhyk

frankmcsherry added some commits Apr 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.