This is an automated email from the ASF dual-hosted git repository.
dockerzhang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/inlong.git
The following commit(s) were added to refs/heads/master by this push:
new 175763e85a [INLONG-11190][SDK] Optimize Transform SDK Translate
Function (#11191)
175763e85a is described below
commit 175763e85aea64a9ddc073c7f0d6ad0391f0208d
Author: vernedeng <[email protected]>
AuthorDate: Tue Sep 24 12:04:04 2024 +0800
[INLONG-11190][SDK] Optimize Transform SDK Translate Function (#11191)
---
.../inlong/sdk/transform/process/function/TranslateFunction.java | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git
a/inlong-sdk/transform-sdk/src/main/java/org/apache/inlong/sdk/transform/process/function/TranslateFunction.java
b/inlong-sdk/transform-sdk/src/main/java/org/apache/inlong/sdk/transform/process/function/TranslateFunction.java
index a51cbad7e4..4953ac9b63 100644
---
a/inlong-sdk/transform-sdk/src/main/java/org/apache/inlong/sdk/transform/process/function/TranslateFunction.java
+++
b/inlong-sdk/transform-sdk/src/main/java/org/apache/inlong/sdk/transform/process/function/TranslateFunction.java
@@ -48,7 +48,7 @@ import java.util.Map;
* find_chars: A string containing the characters to be replaced.
* replace_chars: A string containing the characters to substitute.
* examples:
- * case1: translate(email, '@', '.') -> original_expression:
[email protected] target_expression: harry.inlong.com
+ * case1: translate([email protected], '@', '.') -> original_expression:
[email protected] target_expression: harry.inlong.com
* case2: translate(hello WorD, 'WD', 'wd') -> original_expression: hello
WorD target_expression: hello word
*/
@TransformFunction(names = {"translate"})
@@ -108,9 +108,7 @@ public class TranslateFunction implements ValueParser {
return ImmutableMap.of();
}
- final int findSize = findChars == null ? 0 : findChars.length();
- final int replaceSize = replaceChars == null ? 0 :
replaceChars.length();
- final int commonSize = Math.min(findSize, replaceSize);
+ final int commonSize = Math.min(findChars.length(),
replaceChars.length());
// Create a map to store character replacements
Map<Character, Character> replacementMap = new HashMap<>();
for (int i = 0; i < commonSize; i++) {
