-
Notifications
You must be signed in to change notification settings - Fork 266
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
Rework commands (and deprecate registration API) #4397
Comments
Hi @fgalan, @mapedraza, Is it correct to understand that this issue requires the following two actions?
If the above understanding is correct, regarding 2., there is a use case that uses registrations, so please do not delete it (To delete the registration API, github issue #4402 has been created). Use case: Advantages of registrations in the above use case:
|
Hi @ArqamFarooqui110719 your understanding is correct, the plans are:
This would mean CB will no longer support registrations ("act as aggregator"), but, in any case, it will be able to act as context provider for another system. For As I addressed in the issue you mention (see #4149 (comment)) we alternatively propose an external component that act as agregator. This component can be developed using other technologies (such as python or nodejs) to ease the development and also be part of the orion repo. Being said that, we are open to listen in deep your uses cases and see how we can approach this |
Commands are currently based on Context Providers (CPr), sending command requests based on context updates forwarding (based on registrations). This mechanism has a lot of problems, as detailed in this issue.
The same sending of command request can be achieved using custom notifications (based on subscriptions). A subscription is a much more convenient way of sending requests to endpoints, widely used and without any existing important issue.
Once the new commands mechanism get implemented in IoT Agents, the registration API (and associated request forwarding) will be deprecated and eventually removed (that's part of issue #4402). Note that registries are also used to forward queries (out of the scope of the command use case) but this functionality is not actually used in relevant use cases, so its value is small and its deprecation/removal doesn't hurt.
The text was updated successfully, but these errors were encountered: