[fix][broker] Fix unexpected behaviour by invoke PulsarConfigurationLoader#convertFrom

[mattisonchao]

Motivation

This regression is introduced by #16234.

I found it by setting proxyAdditionalServlets on the proxy, I expected it could load the additional servlet, but it didn’t work well. the proxyConfig.getProperties().getProperty("proxyAdditionalServlets") method returns null after invoking PulsarConfigurationLoader#convertFrom.

After investigating, The root cause is we use the same properties object when we invoke convertFrom method. And then, it set the unknown field to the new config without any type check.

Modifications

• Don't use the same properties object when converting.

Verifying this change

• Make sure that the change passes the CI checks.

• doc-not-needed

[github-actions]

@mattisonchao Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

[codecov-commenter]

Codecov Report

Merging #18816 (8d69886) into master (68ca60c) will decrease coverage by 8.45%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #18816      +/-   ##
============================================
- Coverage     50.05%   41.60%   -8.46%     
+ Complexity    11024     2404    -8620     
============================================
  Files           703      235     -468     
  Lines         68814    16671   -52143     
  Branches       7378     1819    -5559     
============================================
- Hits          34446     6936   -27510     
+ Misses        30621     8974   -21647     
+ Partials       3747      761    -2986     

Flag	Coverage Δ
unittests	41.60% <ø> (-8.46%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...apache/pulsar/client/impl/AutoClusterFailover.java	69.44% <0.00%> (-0.56%)	⬇️
...ersistentStickyKeyDispatcherMultipleConsumers.java
...r/stats/prometheus/PrometheusMetricsGenerator.java
...er/loadbalance/extensions/data/BrokerLoadData.java
...che/bookkeeper/mledger/impl/ManagedLedgerImpl.java
...ache/pulsar/broker/namespace/NamespaceService.java
...er/mledger/impl/ManagedLedgerFactoryMBeanImpl.java
...rg/apache/pulsar/broker/namespace/OwnedBundle.java
...rvice/nonpersistent/NonPersistentSubscription.java
...he/pulsar/broker/intercept/BrokerInterceptors.java
... and 459 more

[congbobo184]

LGTM! could you please add a test for annotation FieldContext.class