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

publish/subscribe model and "ad-hoc" queries #9

Closed
henkbirkholz opened this issue Jun 29, 2015 · 3 comments
Closed

publish/subscribe model and "ad-hoc" queries #9

henkbirkholz opened this issue Jun 29, 2015 · 3 comments

Comments

@henkbirkholz
Copy link
Member

It seems that SACM targets to include a publish/subscribe model (as a specialization of the provider/consumer model) as a part of the architecture. Is this correct?

If this is indeed intended to be a core concept, the structure of the draft needs improvement, as this concept is barely introduced at the moment and bits and pieces relating to it are scattered throughout the text (sometimes quite surprisingly so). E.g. The term Broker is introduced via a diagram (section 3) and mentioned the first time at the end of section 3.1.3. mid-sentence. The term Subscription (or anything labeled "subscr*") is mentioned only once as a side comment in parentheses, etc.

A potential remedy could be an better aggregated (and early) introduction of the concept in the architecture draft?

If it is not a core concept of the architecture (leaving - as it is - Controller/Provider/Consumer as the remaining core terms), a consolidated paragraph in the text could highlight the relationship to the Publish/Subscribe concept (and maybe refer to IM and DM drafts).

The same applies to "ad-hoc queries", which are only mention implicitly via the term "one-time query" near to the introduction of "subscription" in 3.1.2.

@llorenzin
Copy link

Resolution - add a sentence to 3 to describe the high-level SACM operations (publish, query, subscribe).

@llorenzin
Copy link

Fixed in -04 - added to description of functions

@llorenzin
Copy link

Henk to review and close or update

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

No branches or pull requests

2 participants