indhuvisu opened a new pull request, #1190:
URL: https://github.com/apache/incubator-eventmesh/pull/1190
<!--
(The sections below can be removed for hotfixes of typos)
-->
Fixes #<1152>.
### Motivation
ClientGroupWrapper EventMeshProducer classes using the redundant plain
strings as property keys and values, which in turn will cause inconsistency in
the project. In future if any of that String literal needs to be changed/reused
its better to extract and collate them in single EventMeshConstants class,
which will ease the maintainability of code.
### Modifications
Introduced a new constants and also reused few of existing constants from
EventMeshConstants and have replaced the Constant values in ClientGroupWrapper,
EventMeshProducer.
### Documentation
- Does this pull request introduce a new feature? ( no)
- If yes, how is the feature documented? (not applicable)
- If a feature is not applicable for documentation, explain why?
Since the change not introducing any new feature/ functional change,
documentation is not required.
- If a feature is not documented yet in this PR, please create a followup
issue for adding the documentation
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
