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

Overlapping streams - shared events #67

Closed
aisaac opened this issue Feb 19, 2020 · 6 comments
Closed

Overlapping streams - shared events #67

aisaac opened this issue Feb 19, 2020 · 6 comments

Comments

@aisaac
Copy link
Member

aisaac commented Feb 19, 2020

Following an issue raised by @higginsr about the registry, there were discussions on call 2020-01-22 and call 2020-02-19, which clarified that:

  • Multiple streams may contain the same events
  • Processing multiple streams, you may want to merge them into a single stream, rather than doing streams consecutively.

The first of these call concluded that this could be handled via a recipe, which would tell harvesters what they can expect and suggesting the options. In the second, we questioned a potential recipe as potentially quite shallow (since we basically tell harvesters can proceed as they want). We may as well add a line in the spec that says that events can be contained in different streams (this could be seen as a clarification of the semantics for containment of events in streams)

This issue is raised for discussion and as a reminder for implementation.

@aisaac aisaac changed the title Events overlapping Overlapping streams - shared events Feb 19, 2020
@azaroth42 azaroth42 self-assigned this Mar 4, 2020
@azaroth42 azaroth42 added ready and removed discuss labels Mar 5, 2020
@aisaac
Copy link
Member Author

aisaac commented Mar 18, 2020

Cf discussions on calls 2020-03-04 and 2020-03-17

@azaroth42
Copy link
Member

Close?

@aisaac
Copy link
Member Author

aisaac commented Jun 8, 2020

I think that the intro of 3.5 and section 3.5.4 solve the issue. But it would be nice to have the raiser of the issue @higginsr formally confirm!

@higginsr
Copy link

higginsr commented Jun 8, 2020

Yes, I am in a content state

@aisaac
Copy link
Member Author

aisaac commented Jun 9, 2020

@higginsr this is a powerful argument for keeping the current API name in #70 ;-)

@azaroth42
Copy link
Member

Closing, call of 2020-06-10

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

No branches or pull requests

3 participants