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

Rename aloc to eoc #3129

Open
jtfmumm opened this issue Apr 24, 2020 · 0 comments
Open

Rename aloc to eoc #3129

jtfmumm opened this issue Apr 24, 2020 · 0 comments

Comments

@jtfmumm
Copy link
Contributor

jtfmumm commented Apr 24, 2020

aloc shows up in the code in relation to sources, sinks, and connectors. This meant "at least once connectors", but it currently is also used for components that are better described as "effectively once" since they handle both at least once and at most once semantics. We should rename aloc to eoc unless there's a case where the component does not support at most once.

Marked as a bug only because this is misleading and could lead to mistakes down the line.

@jtfmumm jtfmumm changed the title Rename aloc to eo Rename aloc to eoc Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant