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(i18n): desc of strategy in RocketMQ #40

Merged
merged 1 commit into from
May 13, 2024
Merged

fix(i18n): desc of strategy in RocketMQ #40

merged 1 commit into from
May 13, 2024

Conversation

Kinplemelon
Copy link
Member

@@ -281,7 +281,7 @@ export const zhIntegrationDesc: Record<string, Record<string, string>> = {
sync_timeout: 'RocketMQ 驱动同步调用的超时时间。',
namespace:
'如果你正在使用阿里云的 RocketMQ 服务并且启用了命名空间,或者你在你自己的 RocketMQ 服务里配置了命名空间,那么你必须配置命名空间字段。对于阿里云的 RocketMQ 服务来说,命名空间就是实例 ID。',
strategy: 'Produce 策略,注意`Message Key`设置为`none`时,不可选用`key_dispatch`策略',
strategy: '生产者密钥分派策略,默认为 `roundrobin`,支持占位符。',
Copy link
Member

@0721Betty 0721Betty May 13, 2024

Choose a reason for hiding this comment

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

Does it work with all 5.6+ versions of EMQX?

Copy link
Member

Choose a reason for hiding this comment

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

No. based on 5.7

Copy link
Member

Choose a reason for hiding this comment

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

I see, the strategy field is added from v5.7
so, it will not affect the old EMQX

Copy link
Member

Choose a reason for hiding this comment

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

yes!

@ysfscream ysfscream merged commit 0ca1589 into main May 13, 2024
1 check passed
@ysfscream ysfscream deleted the kinple/dev branch May 13, 2024 09:59
@Kinplemelon Kinplemelon restored the kinple/dev branch May 16, 2024 01:40
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.

3 participants