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

[ISSUE #159] [Doc] The introductions of EventMesh Connector and EventMesh Storage are incorrect #232

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Whitegabriella789
Copy link

Fixes #159

Motivation

Explain the content here.

The content of the EventMesh Runtime documentation was outdated and contained incorrect references to the EventMesh Connector. The documentation needed updates to ensure accurate instructions for configuring EventMesh Storage, as well as correcting/checking image paths and terminology.

Explain why you want to make the changes and what problem you're trying to solve.

The changes were made to address user confusion regarding the correct setup and usage of EventMesh Storage and Connectors, as highlighted in issues #159 and #4680. This ensures users can correctly configure and deploy EventMesh with RocketMQ as the storage layer.

Modifications

Describe the modifications you've done.

  • Updated all references from "EventMesh Connector" to "EventMesh Storage" where applicable.

  • Checked the image path to make sure it started with /static/images/ so images are correctly displayed.

  • Revised configuration instructions to specify eventMesh.storage.plugin.type=rocketmq instead of eventMesh.connector.plugin.type.

  • Added a note at the bottom to clarify the distinction between EventMesh Connectors and EventMesh Storage, including their deployment and configuration.

Do let me know if there's anything that needs fixing or if I understood the instructions wrong. Thanks!

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? not applicable
  • If a feature is not applicable for documentation, explain why? not applicable
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation. not applicable

Ensured image source was corrected. Also switched Event Store to EventMesh Storage (which I believe is what was requested). If there's anything else that needs correcting or I understood the instructions wrong, do let me know, and I'll be glad to fix it.
@Pil0tXia Pil0tXia added the ready for review PR is waiting for reviewer's approval or opinion (used as a strong reminder) label Jul 11, 2024
Comment on lines +3 to 5
EventMesh Runtime is a stateful Mesh node in the EventMesh cluster, responsible for event transmission between Source Connectors and Sink Connectors. It uses EventMesh Storage as a storage queue for events.

![EventMesh Runtime](../../static/images/design-document/runtime.png)
Copy link
Member

Choose a reason for hiding this comment

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

The term "Event Store" has been uniformly used in the Eventmesh-site documentation and architecture diagrams, which you can discover by reading the git history.

The main purpose of the issue is to correct the deployment and use of the connector and storage-plugin. You can try deploying a pair of connectors yourself and fix any outdated or incorrect parts in the documentation.

Copy link
Member

Choose a reason for hiding this comment

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

  • Updated all references from "EventMesh Connector" to "EventMesh Storage""Event Store" where applicable.

  • Checked the image path to make sure it started with /static/images/ so images are correctly displayed.

  • Revised configuration instructions to specify eventMesh.storage.plugin.type=rocketmq instead of eventMesh.connector.plugin.type.

I haven't seen any modifications regarding these points. Could you please check if they have been pushed onto git? Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review PR is waiting for reviewer's approval or opinion (used as a strong reminder)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Doc] The introductions of EventMesh Connector and EventMesh Storage are incorrect
2 participants