Skip to content

[AMORO-3713] fix bug of merging into when selecting less columns from souce table than target mixed table#3722

Merged
Aireed merged 1 commit intoapache:masterfrom
nicochen:fix_indexOutOfB_merge_into_bug_when_partial_update
Aug 13, 2025
Merged

[AMORO-3713] fix bug of merging into when selecting less columns from souce table than target mixed table#3722
Aireed merged 1 commit intoapache:masterfrom
nicochen:fix_indexOutOfB_merge_into_bug_when_partial_update

Conversation

@nicochen
Copy link
Contributor

@nicochen nicochen commented Aug 8, 2025

Why are the changes needed?

Close #3713.

Brief change log

  • Revise the logic of fetching projection rows and sourceOutput

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable)

@github-actions github-actions bot added the module:mixed-spark Spark module for Mixed Format label Aug 8, 2025
@Aireed Aireed force-pushed the fix_indexOutOfB_merge_into_bug_when_partial_update branch from 22e091c to a40dd8d Compare August 12, 2025 01:14
@Aireed
Copy link
Contributor

Aireed commented Aug 12, 2025

@baiyangtx PTAL.

@Aireed Aireed merged commit 28a6415 into apache:master Aug 13, 2025
5 of 7 checks passed
Jzjsnow pushed a commit to Jzjsnow/amoro that referenced this pull request Aug 22, 2025
… souce table than target mixed table (apache#3722)

[AMORO-3713]got IndexOutOfBoundsException When Using Partial Table Fields as MERGE Source

(cherry picked from commit 28a6415)
zhoujinsong pushed a commit that referenced this pull request Aug 25, 2025
… souce table than target mixed table (#3722)

[AMORO-3713]got IndexOutOfBoundsException When Using Partial Table Fields as MERGE Source

(cherry picked from commit 28a6415)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:mixed-spark Spark module for Mixed Format

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: got IndexOutOfBoundsException When Using Partial Table Fields as MERGE Source

2 participants