Skip to content

branch-4.0: [fix](function) Fix array_join to handle non-constant columns. #60612#60703

Closed
github-actions[bot] wants to merge 1 commit intobranch-4.0from
auto-pick-60612-branch-4.0
Closed

branch-4.0: [fix](function) Fix array_join to handle non-constant columns. #60612#60703
github-actions[bot] wants to merge 1 commit intobranch-4.0from
auto-pick-60612-branch-4.0

Conversation

@github-actions
Copy link
Contributor

Cherry-picked from #60612

The previous implementation of array_join was incorrect: it didn't
consider whether the inputs were constant columns and always took the
first element from each column.

 before
```sql
mysql> select * from db_array_join; 
+------+-----------------+-------+
| id   | sarr            | s     |
+------+-----------------+-------+
|    1 | ["a", "b", "c"] | 0000  |
|    2 | ["c", "d", "e"] | 11111 |
+------+-----------------+-------+

mysql> select array_join(sarr,s) from db_array_join;
+--------------------+
| array_join(sarr,s) |
+--------------------+
| a0000b0000c        |
| c0000d0000e        |
+--------------------+
```
now
```sql
mysql> select array_join(sarr,s) from db_array_join;
+--------------------+
| array_join(sarr,s) |
+--------------------+
| a0000b0000c        |
| c11111d11111e      |
+--------------------+
```
@github-actions github-actions bot requested a review from yiguolei as a code owner February 12, 2026 05:46
@Thearas
Copy link
Contributor

Thearas commented Feb 12, 2026

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@dataroaring dataroaring reopened this Feb 12, 2026
@Thearas
Copy link
Contributor

Thearas commented Feb 12, 2026

run buildall

@Mryange
Copy link
Contributor

Mryange commented Feb 12, 2026

有前置pr。
我在#60707
里面手动pick,这个可以close掉了。

@yiguolei yiguolei closed this Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants