Trim config values before binding properties to objects

[AliasJeff]

Which Issue(s) This PR Fixes

Fixes #10708

Brief Description

This PR trims property values before binding them to config objects in MixAll.properties2Object.

Previously, only String values were trimmed. Numeric and boolean values were parsed as-is, so a value like fileReservedTime = 168 failed to parse and the config field kept its default value.

This change trims the value before type conversion, so int, long, double, float, boolean, and String values are handled consistently.

How Did You Test This Change?

Added tests for:

• primitive config values with leading/trailing whitespace;
• fileReservedTime = 168 being parsed as 168 instead of keeping the default value.

Test commands:

mvn -pl common -DskipITs -Dspotbugs.skip=true -Dcheckstyle.skip=true -Dlicense.skip=true -Dtest=org.apache.rocketmq.common.UtilAllTest#testProperties2ObjectTrimsPrimitiveValues test
mvn -pl broker -am -DskipITs -Dspotbugs.skip=true -Dcheckstyle.skip=true -Dlicense.skip=true -DfailIfNoTests=false -Dtest=org.apache.rocketmq.broker.BrokerControllerTest#testFileReservedTimeWithTrailingWhitespaceIsPreserved test

Both targeted tests passed locally.

[RockteMQ-AI]

Summary

Fixes a real bug where config properties with leading/trailing whitespace silently fail to parse for non-String types (int, long, double, float, boolean). Moving property.trim() before the type-switch block is the correct and minimal fix.

Changes reviewed:

• MixAll.properties2Object() — trim moved before type dispatch (correct)
• UtilAllTest — comprehensive test covering all primitive types with whitespace
• BrokerControllerTest — specific regression test for the fileReservedTime = 168 scenario from #10708

LGTM. Clean fix with good test coverage.

---

Automated review by "github-manager-bot"