TheNorthMemory commented on issue #5768:
URL: https://github.com/apache/rocketmq/issues/5768#issuecomment-1489633516

   Searching the github with [**GET_KV_CONFIG_BY_VALUE with language 
Java**](https://github.com/search?l=Java&q=GET_KV_CONFIG_BY_VALUE&type=Code), 
there were a lots of `public static final int GET_KV_CONFIG_BY_VALUE = 217;` 
definitions on the rocketmq previous versions.  The `GET_KV_CONFIG_BY_VALUE` is 
not a new `RequestCode`, It was already exists.
   
   I think the code **217** was inappropriated mapping to 
`REGISTER_TOPIC_IN_NAMESRV` since **5.0.0-beta** and sugguest that creating a 
new code, eg: something like `1217` for the `REGISTER_TOPIC_IN_NAMESRV`.
   
   @Abhijeetmishr 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to