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

Initial take at supporting topics in messaging #4956

Closed
wants to merge 3 commits into from

Conversation

derekperkins
Copy link
Member

@derekperkins derekperkins commented Jun 26, 2019

This isn't functional, but should describe well enough the approach I am suggesting. With minimal changes to the existing flow, it stores a map of topics on the engine. Then on inserts, it checks to see if the table name references a topic. If so, it recursively replaces the insert statement table with each of the subscribed message table names. Topics are insert only. Updates and deletes only happen on the individual message tables.

Fixes #4940

Signed-off-by: Derek Perkins <derek@derekperkins.com>
Signed-off-by: Derek Perkins <derek@derekperkins.com>
Signed-off-by: Derek Perkins <derek@derekperkins.com>
@derekperkins
Copy link
Member Author

closing in favor of #5011

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

Successfully merging this pull request may close these issues.

Proposal: add topic abstraction to messaging
1 participant