snicoll opened a new issue #8: `RocketMQProperties#nameServer` should offer a 
more user-friendly format
URL: https://github.com/apache/rocketmq-spring/issues/8
 
 
   The `nameServer` property ultimately maps to `ClientConfig#setNamesrvAddr` 
that requires a `host:port` pair separated by `;`
   
   I think it would be nicer to not expose that complexity back to the 
configuration. Rather, the configuration should expose a `List<String>` where 
each item represent a `host:port` pair.
   
   There are several advantages to this:
   
   * Each pair can be checked individually and an exception can be thrown if 
its format is invalid
   * YAML configuration is much more readable as you can use the native list 
format
   
   ```yml
   spring:
     rocketmq:
       name-server:
         - 10.0.0.1:9876
         - 10.0.0.2:9876
   ```
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to