Skip to content

Mailbox 211 : distributed event systems #12

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.

Sign up for GitHub

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

Jump to bottom
Closed
chibenwa wants to merge 21 commits into from
Closed

Mailbox 211 : distributed event systems #12

chibenwa wants to merge 21 commits into from

Conversation

@chibenwa
Copy link
Contributor

@chibenwa chibenwa commented Oct 30, 2015

In this PR I propose 3 kind of event system :

  • the default one : the previous one, revisited. Better failure handling, and package relocation
  • a broadcast based event system : Well if you have global mailbox listeners that will not perform well in a distributed environment.
  • a registered based event system : the scalability limit is the numbers of topics you can handle on your publish subscribe system. It does not support non distributed global Listeners.

Note that the on going discussion about Message Sequence Number handling via the event system is still an open debate ;-)

chibenwa added 21 commits November 4, 2015 12:36
@chibenwa
MAILBOX-211 Indent CacheInvalidatingMailboxListener with 4 spaces
fd1e700
@chibenwa
MAILBOX-211 MailboxListener should indicate its type
ae96d5a
@chibenwa
MAILBOX-211 Moving event related interfaces and code to a new package
38cc1e7
@chibenwa
MAILBOX-211 Rewrite Default DelegatingMailboxListener
5bf40ab 
 - Rely on composition rather than on inheritance for capabilities extension
 - Generalise it so that working on on the event system will be easier
 - Rely on well tested Guava stuff instead of hand craffted collections
@chibenwa
MAILBOX-211 Introduce event factories
ed11e15
@chibenwa
MAILBOX-211 Introduce MailboxId de serializer
95e66cd
@chibenwa
MAILBOX-211 Move SimpleMailboxACLJsonConverterTest to a new package
e0fec71
@chibenwa
MAILBOX-211 Event serialization
a06146f
@chibenwa
MAILBOX-211 Interfaces abstracting a publish subscribe system
f0902d5
@chibenwa
MAILBOX-211 Add a registration system based on MailboxPaths and a Cas…
2ea5e16 
…sandra based implementation
@chibenwa
MAILBOX-211 Event processing should not be stopped by error thrown
f70f7f6
@chibenwa
MAILBOX-211 Distributed Delegating Mailbox Listener and a registered …
ce5cb28 
…based implementation
@chibenwa
MAILBOX-211 Adding a Broadcast based event system
1198021
@chibenwa
MAILBOX-211 Broadcast integration test
ee4c36b
@chibenwa
MAILBOX-211 Integration test for Cassandra based distributed event sy…
a4aad5c 
…stem.
@chibenwa
MAILBOX-211 Adding Kafka publishing system
c363ed5
@chibenwa
MAILBOX-211 Mailbox Id deserializer should be Spring defined
ffeee6c
@chibenwa
MAILBOX-211 Add a configuration reader for Spring about the Event system
c850341
@chibenwa
MAILBOX-211 Event system documentation
b311438
@chibenwa
inc max retry
2a8daf0
@chibenwa
Increase timeout
5f58c43
@chibenwa
Copy link
Contributor Author

chibenwa commented Nov 4, 2015

I updated the PR after Matthieu comments.

It still needs to be unit test validated ( will take ~ an hour ).

@chibenwa chibenwa closed this Nov 28, 2015
@mbaechler mbaechler mentioned this pull request Oct 17, 2019
Closed
asfgit pushed a commit that referenced this pull request Apr 22, 2020
@chibenwa
JAMES-2997 step #12 Attachment builder should require attachmentId
0f4142e 
It should not be set implicitly
@chibenwa chibenwa mentioned this pull request Sep 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@chibenwa