Core, Open-API: Use String.replace for literal replacements

[uros-b]

Several places used String.replaceAll with literal (non-regex) patterns, which compiles a regex Pattern on every call. Switch these to the char/CharSequence String.replace overloads, which avoid the per-call compilation. Behavior-identical.

[uros-b]

Waiting for CI, cc @szehon-ho for review

[ebyhr]

Could you confirm other replaceAll usages? e.g. ReportMetricsRequestParser, RCKUtils

[uros-b]

Good catch, thank you @ebyhr!

[ebyhr]

We can update more:

iceberg/spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/procedures/CreateChangelogViewProcedure.java

Line 297 in 58d5c37

return "`" + columnName.replaceAll("`", "``") + "`";

iceberg/spark/v4.1/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java

Line 806 in 58d5c37

TableIdentifier.of("default", QUOTED_SPECIAL_CHARS_TABLE_NAME.replaceAll("`", ""));

iceberg/spark/v4.1/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java

Line 844 in 58d5c37

TableIdentifier.of("default", QUOTED_SPECIAL_CHARS_TABLE_NAME.replaceAll("`", ""));

iceberg/spark/v4.1/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java

Line 887 in 58d5c37

TableIdentifier.of("default", QUOTED_SPECIAL_CHARS_TABLE_NAME.replaceAll("`", ""));