[ISSUES #3482]Replace this call to "replaceAll()" by a call to the "replace()" method.[ProducerImpl]

[harshithasudhakar]

Fixes #3482

Motivation

When String::replaceAll is used, the first argument should be a real regular expression. If it’s not the case, String::replace does exactly the same thing as String::replaceAll without the performance drawback of the regex.

Modifications

Replaced "replaceAll()" to "replace()"

Documentation

• Does this pull request introduce a new feature? (yes / no)
  No

[Alonexc]

LGTM

[codecov]

Codecov Report

Merging #3631 (bf6992c) into master (4eb819f) will not change coverage.
The diff coverage is 50.00%.

❗ Current head bf6992c differs from pull request most recent head dba92da. Consider uploading reports for the commit dba92da to get more accurate results

@@            Coverage Diff            @@
##             master    #3631   +/-   ##
=========================================
  Coverage     13.76%   13.76%           
  Complexity     1291     1291           
=========================================
  Files           571      571           
  Lines         29239    29239           
  Branches       2885     2885           
=========================================
  Hits           4024     4024           
  Misses        24841    24841           
  Partials        374      374           

Impacted Files	Coverage Δ
...ntmesh/storage/rocketmq/producer/ProducerImpl.java	21.64% <50.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more