-
Notifications
You must be signed in to change notification settings - Fork 403
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
Design new StatusPort
#982
Comments
@mossmaurice maybe I missed something but the purpose of the interface port is to forward capro messages for instance to gateway ports. And shouldn't it be called StatePort instead of StatusPort? |
That's correct. More broadly speaking
I think that's a misunderstanding. I'm talking about these |
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
… and clarify what a Transaction is Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
…d remove wrong description of memory order Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
StatusPort
StatusPort
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
…ood idea Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Not needed anymore. Closing. |
Brief feature description
After this discussion the conclusion was a
StatusPort
orBroadcastPorts
is beneficial for #415 and for the introspection topics, in order to prevent any user totake()
the data and notrelease()
which could lead to a out-of-memory error.Detailed information
StatusPort
ImplementMoved to ImplementStatusPort
StatusPort
#1096SendMoved to ImplementServiceRegistry
viaStatusPort
s (see service registry as a built-in topic #415)StatusPort
#1096ReplaceMoved to ImplementInterfacePort
s withStatusPort
sStatusPort
#1096References
The text was updated successfully, but these errors were encountered: