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

Fix the bug of Get Start in Readme #78

Merged
merged 2 commits into from
Apr 20, 2022
Merged

Fix the bug of Get Start in Readme #78

merged 2 commits into from
Apr 20, 2022

Conversation

DongyuanPan
Copy link
Contributor

#77

Copy link
Contributor

@tianliuliu tianliuliu left a comment

Choose a reason for hiding this comment

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

LGTM

@DongyuanPan
Copy link
Contributor Author

btw,add tips for broker configuration

@@ -18,6 +18,11 @@ The queue model of MQTT needs to be based on the light message queue, For the de
The queue model of MQTT needs to be based on the light message queue feature ([RIP-28](https://github.com/apache/rocketmq/pull/3694)) of RocketMQ. RocketMQ has only supported this feature since version 4.9.3. Please ensure that the installed version of RocketMQ already supports this feature.

For the quick start of light message queue, please refer to [Example_LMQ](https://github.com/apache/rocketmq/blob/develop/docs/cn/Example_LMQ.md) of RocketMQ.
For example, set the following parameters to true in broker.conf
Copy link
Contributor

Choose a reason for hiding this comment

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

good job!!!

Copy link
Member

@ShannonDing ShannonDing left a comment

Choose a reason for hiding this comment

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

LGTM

@ShannonDing ShannonDing merged commit 344cb70 into apache:main Apr 20, 2022
@ShannonDing ShannonDing linked an issue Apr 20, 2022 that may be closed by this pull request
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.

Fix the bug of Get Start in Readme
3 participants