[BugFix] append_trailing_char_if_absent return empty string for const empty strings

[fzhedu]

Signed-off-by: fzhedu [email protected]

What type of PR is this：

• BugFix
• Feature
• Enhancement
• Refactor
• UT
• Doc
• Tool

Which issues of this PR fixes ：

Fixes ##12769

Problem Summary(Required) ：

append_trailing_char_if_absent's behavior is different when the first arg is an empty string from columns or not.

mysql> Create table t ( a int, name char(20), pv double) ENGINE=OLAP DUPLICATE KEY(a)  DISTRIBUTED BY HASH(a) BUCKETS 10 PROPERTIES("replication_num" = "1");
Query OK, 0 rows affected (0.01 sec)

mysql> insert into t values (1,'',3);
Query OK, 1 row affected (0.03 sec)
{'label':'insert_a4e0c3d6-670e-11ed-b25b-00163e1f2aee', 'status':'VISIBLE', 'txnId':'5640284'}

mysql> select append_trailing_char_if_absent(name,'a') from t;
+-------------------------------------------+
| append_trailing_char_if_absent(name, 'a') |
+-------------------------------------------+
|                                           |
+-------------------------------------------+
1 row in set (0.01 sec)


mysql> select append_trailing_char_if_absent('','a') from t;
+-----------------------------------------+
| append_trailing_char_if_absent('', 'a') |
+-----------------------------------------+
| a                                       |
+-----------------------------------------+
1 row in set (0.00 sec)

Checklist:

• I have added test cases for my bug fix or my new feature
• This pr will affect users' behaviors
• This pr needs user documentation (for new or modified features or behaviors)
  • I have added documentation for my new feature or new function

[github-actions]

Please check the version labels which this pr will be auto backported to target branch and click the checkbox:

• check done.

[wanpengfei-git]

run starrocks_admit_test

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
1.1% Duplication

[fzhedu]

run starrocks_be_unittest

[fzhedu]

@mergify backport-2.2

[fzhedu]

@mergify backport-2.3

[fzhedu]

@mergify backport-2.4

[fzhedu]

@mergify backport-2.5

[mergify]

backport -2.2

❌ No backport have been created

• Backport to branch -2.2 failed: Branch not found

[mergify]

backport -2.3

❌ No backport have been created

• Backport to branch -2.3 failed: Branch not found

[mergify]

backport -2.4

❌ No backport have been created

• Backport to branch -2.4 failed: Branch not found

[mergify]

backport -2.5

❌ No backport have been created

• Backport to branch -2.5 failed: Branch not found

[fzhedu]

@mergify backport branch-2.5

[fzhedu]

@mergify backport branch-2.4

[fzhedu]

@mergify backport branch-2.3

[fzhedu]

@mergify backport branch-2.2

[mergify]

backport branch-2.5

✅ Backports have been created

• #13781 [BugFix] append_trailing_char_if_absent return empty string for const empty strings (backport #13762) has been created for branch branch-2.5

[mergify]

backport branch-2.4

✅ Backports have been created

• #13782 [BugFix] append_trailing_char_if_absent return empty string for const empty strings (backport #13762) has been created for branch branch-2.4

[mergify]

backport branch-2.3

✅ Backports have been created

• #13783 [BugFix] append_trailing_char_if_absent return empty string for const empty strings (backport #13762) has been created for branch branch-2.3

[mergify]

backport branch-2.2

✅ Backports have been created

• #13784 [BugFix] append_trailing_char_if_absent return empty string for const empty strings (backport #13762) has been created for branch branch-2.2