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

Update Architecture page in Developer's Guide #396

Merged
merged 2 commits into from
Oct 27, 2023

Conversation

sarina
Copy link
Contributor

@sarina sarina commented Oct 24, 2023

Update the architecture page in the Dev guide

Update the diagram to remove old components and add MFEs

Source for the diagram can be found on Miro: https://miro.com/app/board/uXjVNVzVLPM=/?share_link_id=276497217755 - anyone from Axim can edit this, and it should be commentable by anyone in the community.

Copy link
Contributor

@feanil feanil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This text changes look good, for the diagram, I would:

  • Remove XQueue
  • Replace Insight & Analytics with Aspects
  • Remove Otto
  • Replace RabbitMQ with Redis
  • Replace Hadoop etc, with Clickhouse

@sarina
Copy link
Contributor Author

sarina commented Oct 25, 2023

I updated the diagram ... do we want to represent MFEs anywhere?

@feanil
Copy link
Contributor

feanil commented Oct 25, 2023

@sarina good call, it would be useful to add MFEs in the top gray bar I think since they're kind of above all the backends.

@sarina sarina changed the title Update Architecture page Update Architecture page in Developer's Guide Oct 27, 2023
@sarina sarina force-pushed the sarina/quick-arch-diagram-fix branch from 115f552 to 46a7647 Compare October 27, 2023 01:07
@sarina sarina merged commit f4f124e into main Oct 27, 2023
@sarina sarina deleted the sarina/quick-arch-diagram-fix branch October 27, 2023 01:11
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.

2 participants