Skip to content

Conversation

@CavemanIV
Copy link
Contributor

What changes were proposed in this pull request?

port to 3.5 for [SPARK-43242][CORE] Fix throw 'Unexpected type of BlockId' in shuffle corruption diagnose

Why are the changes needed?

3.5 conflict with PR in master, see end of discussion #40921

Does this PR introduce any user-facing change?

No

How was this patch tested?

Existing tests

…' in shuffle corruption diagnose

A minor bugfix in `ShuffleBlockFetcherIterator.diagnose`, which not handle type ShuffleBlockBatchId properly

`.diagnose()` is used in exception handling try-catch block, throw new exception due to type mismatch (in this case, `ShuffleBlockBatchId`) will swallow original exception stack

No

Existing tests

Closes apache#40921 from CavemanIV/SPARK-43242.

Lead-authored-by: zhangliang <[email protected]>
Co-authored-by: sychen <[email protected]>
Signed-off-by: Yi Wu <[email protected]>
@github-actions github-actions bot added the CORE label Aug 28, 2024
@CavemanIV CavemanIV changed the title port to 3.5 [SPARK-43242][CORE] Fix throw 'Unexpected type of BlockId' in shuffle corruption diagnose [SPARK-43242][CORE][3.5] Fix throw 'Unexpected type of BlockId' in shuffle corruption diagnose Aug 29, 2024
Ngone51 pushed a commit that referenced this pull request Aug 30, 2024
…uffle corruption diagnose

#### What changes were proposed in this pull request?
port to 3.5 for [[SPARK-43242](https://issues.apache.org/jira/browse/SPARK-43242)][CORE] Fix throw 'Unexpected type of BlockId' in shuffle corruption diagnose

#### Why are the changes needed?
3.5 conflict with PR in master, see end of discussion #40921

#### Does this PR introduce any user-facing change?
No

#### How was this patch tested?
Existing tests

Closes #47910 from CavemanIV/port3.5-SPARK-43242.

Authored-by: zhangliang <[email protected]>
Signed-off-by: Yi Wu <[email protected]>
@Ngone51
Copy link
Member

Ngone51 commented Aug 30, 2024

Thanks, merged to branch-3.5.

@Ngone51 Ngone51 closed this Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants