[
https://issues.apache.org/jira/browse/ROCKETMQ-257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124423#comment-16124423
]
ASF GitHub Bot commented on ROCKETMQ-257:
-----------------------------------------
Github user shroman commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/144#discussion_r132808594
--- Diff: common/src/main/java/org/apache/rocketmq/common/MixAll.java ---
@@ -59,7 +59,7 @@
public static final String WS_DOMAIN_NAME =
System.getProperty("rocketmq.namesrv.domain", DEFAULT_NAMESRV_ADDR_LOOKUP);
public static final String WS_DOMAIN_SUBGROUP =
System.getProperty("rocketmq.namesrv.domain.subgroup", "nsaddr");
// // http://jmenv.tbsite.net:8080/rocketmq/nsaddr
-// public static final String WS_ADDR = "http://"; + WS_DOMAIN_NAME +
":8080/rocketmq/" + WS_DOMAIN_SUBGROUP;
+ public static final String WS_ADDR = "http://"; + WS_DOMAIN_NAME +
":8080/rocketmq/" + WS_DOMAIN_SUBGROUP;
--- End diff --
Is it legal to bring `WS_ADDR`? It was commented out for a reason.
> name server address and web server address should be specified at least one
> ---------------------------------------------------------------------------
>
> Key: ROCKETMQ-257
> URL: https://issues.apache.org/jira/browse/ROCKETMQ-257
> Project: Apache RocketMQ
> Issue Type: Bug
> Components: rocketmq-client
> Affects Versions: 4.1.0-incubating
> Environment: test and production
> Reporter: Eason Chen
> Assignee: Xiaorui Wang
> Priority: Minor
> Fix For: 4.2.0-incubating
>
>
> if name server address and web server address both are not specified , client
> will not fetch the
> right name server and client will start fail, because the default
> wsAddr=http://jmenv.tbsite.net:8080/rocketmq/nsaddr is not reachable.
> {code:java}
> // name server address and web server address should be specified at least one
> if (null == this.clientConfig.getNamesrvAddr() &&
> MixAll.getWSAddr().equals(MixAll.WS_ADDR)) {
> throw new MQClientException("name server address and
> web server address should be specified at least one.", null);
> } else if (null == this.clientConfig.getNamesrvAddr()) {
> this.mQClientAPIImpl.fetchNameServerAddr();
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
