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

ONANYCHANGE subscriptions #350

Closed
fgalan opened this issue May 7, 2014 · 16 comments
Closed

ONANYCHANGE subscriptions #350

fgalan opened this issue May 7, 2014 · 16 comments
Assignees
Labels
Milestone

Comments

@fgalan
Copy link
Member

fgalan commented May 7, 2014

NGSI standard ONCHANGE subscription always need a list of "triggering" attributes (see section 5.5.10 in the OMA spec). Thus, it is not possible to specify that the subscriber wants to be notified each time any of the attributes of the entities change without a previous knowledge of the list of attributes of the entity. This is a limitation on some scenarios.

This issue is about to enhance the ContextBroker, including a new type of subscriptions for changes in any attribute (let's name them ONANYCHANGE). In this case the condValue element is not required (this fits in NGSI spec, as this element is optional in subscribeContextRequests according to OMA spec).

Regarding implementation, it would be a matter of tuning the query done to MongoDB in the addTriggeredSubscriptions() function at mongoBackend. It also will involve changes in the processConditionVector(), taking into account the new subscription change (and using the corresponding query for that).

@fgalan fgalan added the P6 label May 7, 2014
@fgalan fgalan added the backlog label Jul 10, 2014
@fgalan fgalan removed the P6 label Oct 15, 2014
@jcballesteros
Copy link

+1

6 similar comments
@ImanolBarba
Copy link

+1

@alexferreiros
Copy link

+1

@goofyahead
Copy link

+1

@elchudi
Copy link

elchudi commented Mar 30, 2015

+1

@dmoranj
Copy link
Contributor

dmoranj commented Sep 23, 2015

+1

@frbattid
Copy link
Member

+1

@silva96
Copy link

silva96 commented Oct 20, 2015

👍

3 similar comments
@c-castillo
Copy link

+1

@dimh
Copy link

dimh commented Oct 20, 2015

+1

@diogonunes92
Copy link

+1

@fgalan fgalan added this to the 0.27.0 milestone Jan 14, 2016
@fgalan fgalan self-assigned this Jan 15, 2016
@fgalan
Copy link
Member Author

fgalan commented Jan 15, 2016

Implemented (finally!) in PR #1679. Once that PR gets merged, it will be available in develop branch. In addition, it would be available in the next official release (0.27.0), which is expected by the end of January / beginning of February.

@fgalan fgalan closed this as completed Jan 15, 2016
@goofyahead
Copy link

Yay! 💃

@fgalan fgalan modified the milestones: QATestPending, 0.27.0 Mar 10, 2016
@fgalan fgalan assigned iariasleon and unassigned fgalan Mar 10, 2016
@fgalan
Copy link
Member Author

fgalan commented Mar 10, 2016

QA Acceptance Test pending. Reopened and asigned to QATestPending milestone.

@fgalan fgalan reopened this Mar 10, 2016
@iariasleon iariasleon modified the milestones: 0.27.0, QATestPending Sep 26, 2016
@iariasleon
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests