-
Notifications
You must be signed in to change notification settings - Fork 104
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 guard for initial notifications #2044
Comments
Similar to issue telefonicaid/perseo-fe#526 |
Hey @fgalan, Anyway, I came across the issue on orion, about Initial notifications being deprecated, and since its mentioned here that "if you need to know the status of your system at subscription time, then use GET /v2/entities with proper pagination." i have a doubt. I understand the reasons for deprecating this feature. (Or maybe im misunderstanding the use of cygnus?) |
I understand that 100k refers to the payload size of the entities, no the number of the entities. Initial notification always included as much as 20 entities (see telefonicaid/fiware-orion#591). However, initial notification feature has been already removed in master (so in next version, Orion 3.2.0 will not be) so talking about this is probably not needed :)
The usual usage case in Cygnus is to persist history, i.e. changes in the entities. The initial status of the system is not considered history, from the point of view of Cygnus, only the changes since the feeding subscription is created. In fact, current Cygnus implementation discard the initial notification and pay attention only to regular (non-initial) notifications. However, if you need to set an "initial change" in your storing system (Postgresql in your case) for every entity in your system, you have two alternatives:
|
Initial notifications are going to be deprecated in Orion Context Broker. They are being deprecated in Orion 3.1.0 and the removal is expected for 3.2.0.
When that moment comes, the Cygnus guard code to "protect" against initial notification will no be longer needed.
This PR can be an useful reference: #2043
The text was updated successfully, but these errors were encountered: