[GitHub] zookeeper issue #624: ZOOKEEPER-3108:use a new property server.id in the zoo...

[maoling]

Github user maoling commented on the issue:

    https://github.com/apache/zookeeper/pull/624
  
    @lvfangmin 
    read the 
[doc](http://zookeeper.apache.org/doc/r3.5.1-alpha/zookeeperReconfig.html) and 
the code in the `QuorumPeerConfig.parseDynamicConfig()`,IMO,the` server.id` is 
only to be one of the static config and not included into the  dynamic config 
file.WDYT?


---