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

Remove and clean up fallback sender implementations #5455

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

msimberg
Copy link
Contributor

Remove fallbacks that are removed in P2300 that were still there in later revisions of P0443 (e.g. 14). Specifically:

  • completely removes submit
  • makes execute a "plain" algorithm, i.e. removes the fallback using submit, it only dispatches through tag_dispatch overloads
  • removes the fallback of connect using as_operation etc.
  • removes the fallback of schedule using as_sender, connect, etc.

@msimberg msimberg added type: cleanup category: senders/receivers Implementations of the p0443r14 / p2300 + p1897 proposals labels Jul 16, 2021
@msimberg msimberg added this to the 1.8.0 milestone Jul 16, 2021
hkaiser
hkaiser previously approved these changes Jul 16, 2021
Copy link
Member

@hkaiser hkaiser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! This greatly simplifies things, very nice.

@msimberg
Copy link
Contributor Author

msimberg commented Jul 20, 2021

I think this should be ready to go now, but I'd like to give #5283 priority to be merged first.

@msimberg msimberg changed the title Remove and cleanup fallback sender implementations Remove and clean up fallback sender implementations Jul 30, 2021
@msimberg msimberg force-pushed the executor-scheduler-cleanup branch 2 times, most recently from 5ec8723 to 451a9c4 Compare July 30, 2021 08:26
aurianer
aurianer previously approved these changes Aug 2, 2021
Copy link
Contributor

@aurianer aurianer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks!

@msimberg
Copy link
Contributor Author

msimberg commented Aug 4, 2021

retest cscs

@msimberg msimberg mentioned this pull request Aug 6, 2021
Remove fallbacks that are removed in P2300 that were still there in
later revisions of P0443 (e.g. 14). Specifically:

- completely removes submit
- makes execute a "plain" algorithm, i.e. it does not have a fallback
  using submit, it only dispatches through tag_dispatch overloads
- removes the fallback of connect using as_operation etc.
- removes the fallback of schedule using as_sender, connect, etc.
@msimberg msimberg merged commit f5f1953 into STEllAR-GROUP:master Aug 10, 2021
@msimberg msimberg deleted the executor-scheduler-cleanup branch August 10, 2021 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: senders/receivers Implementations of the p0443r14 / p2300 + p1897 proposals type: cleanup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants