-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add explicit format to magix spec #12
Comments
Even better to use |
Merged
|
SPC-code
pushed a commit
to SciProgCentre/magix-rfc
that referenced
this issue
Jul 29, 2023
SPC-code
pushed a commit
to SciProgCentre/magix-rfc
that referenced
this issue
Jul 29, 2023
SPC-code
pushed a commit
to SciProgCentre/controls-kt
that referenced
this issue
Jul 30, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Based on recent discussions, I suggest making a final change to magix package format:
Add mandatory
format
string field. Which will replace theorigin
field. The format does not point to a specific endpoint, it only specified thepayload
format. Plugins can evaluate messages based on this field.Replace mandatory
origin
by optionaloriginID
(orsourceID
). Which specified the unique ID of a specific endpoint (either system or service).Deprecate
target
field.The reasoning behind those changes is the following:
It is possible to have several connected systems of the same type. For example, there could be two TANGO connectors. In current spec, it is not possible to distinguish between them without creating complex parsing rules for
origin
.We need to be able to identify systems. For example, we will need to be able to subscribe to specific endpoint events. Also we will want to have services that will be able to resolve direct connection from API (like CORBA does in TANGO and LDAP in DOOCS). This direct connection is needed for binary transfer.
Currently there are no cases where the endpoint explicitly creates a message for another endpoint. We are working strictly on subscription policy. Therefore
target
is meaningless.The text was updated successfully, but these errors were encountered: