-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
ARROW-11489: [Rust][DataFusion] Make DataFrame be Send + Sync #9406
Conversation
Thanks for opening a pull request! Could you open an issue for this pull request on JIRA? Then could you also rename pull request title in the following format?
See also: |
I did this in my personal fork and was planning to raise it with Andy. Glad this will be done 👍 |
c1b63cb
to
e25edb1
Compare
fyi @jorgecarleitao / @andygrove / @Dandandan |
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.
LGTM. Thanks for the quick reaction on this!
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.
LGTM 👍
Inspired by a question on the mailing list [link]( https://lists.apache.org/thread.html/r8f81fae08346817fa283804037ed79a4309bb54aa8ed77c354d7baf0%40%3Cuser.arrow.apache.org%3E) Things need to be `Send` + `Sync` in order to be sent between threads (or async tasks). Thus we should make `DataFrame` require `Send` + `Sync` as well so as to be usable in async applications. Closes apache#9406 from alamb/alamb/dataframe_send_sync Authored-by: Andrew Lamb <andrew@nerdnetworks.org> Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
Inspired by a question on the mailing list [link]( https://lists.apache.org/thread.html/r8f81fae08346817fa283804037ed79a4309bb54aa8ed77c354d7baf0%40%3Cuser.arrow.apache.org%3E) Things need to be `Send` + `Sync` in order to be sent between threads (or async tasks). Thus we should make `DataFrame` require `Send` + `Sync` as well so as to be usable in async applications. Closes apache#9406 from alamb/alamb/dataframe_send_sync Authored-by: Andrew Lamb <andrew@nerdnetworks.org> Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
Inspired by a question on the mailing list [link]( https://lists.apache.org/thread.html/r8f81fae08346817fa283804037ed79a4309bb54aa8ed77c354d7baf0%40%3Cuser.arrow.apache.org%3E) Things need to be `Send` + `Sync` in order to be sent between threads (or async tasks). Thus we should make `DataFrame` require `Send` + `Sync` as well so as to be usable in async applications. Closes apache#9406 from alamb/alamb/dataframe_send_sync Authored-by: Andrew Lamb <andrew@nerdnetworks.org> Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
Inspired by a question on the mailing list [link]( https://lists.apache.org/thread.html/r8f81fae08346817fa283804037ed79a4309bb54aa8ed77c354d7baf0%40%3Cuser.arrow.apache.org%3E) Things need to be `Send` + `Sync` in order to be sent between threads (or async tasks). Thus we should make `DataFrame` require `Send` + `Sync` as well so as to be usable in async applications. Closes apache#9406 from alamb/alamb/dataframe_send_sync Authored-by: Andrew Lamb <andrew@nerdnetworks.org> Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
Inspired by a question on the mailing list link
Things need to be
Send
+Sync
in order to be sent between threads (or async tasks). Thus we should makeDataFrame
requireSend
+Sync
as well so as to be usable in async applications.