Skip to content

[SPARK-38462][CORE] Add error class INTERNAL_ERROR_EXECUTOR#41048

Closed
bozhang2820 wants to merge 1 commit intoapache:masterfrom
bozhang2820:spark-38462
Closed

[SPARK-38462][CORE] Add error class INTERNAL_ERROR_EXECUTOR#41048
bozhang2820 wants to merge 1 commit intoapache:masterfrom
bozhang2820:spark-38462

Conversation

@bozhang2820
Copy link
Contributor

What changes were proposed in this pull request?

This change adds a error class INTERNAL_ERROR_EXECUTOR and change exceptions created in the executor package to use that error class.

Why are the changes needed?

This is to move exceptions created in package org.apache.spark.executor to error class.

Does this PR introduce any user-facing change?

Yes. The exceptions created in package org.apache.spark.executor will change to be of error class INTERNAL_ERROR_EXECUTOR.

How was this patch tested?

Existing unit tests.

@github-actions github-actions bot added the CORE label May 4, 2023
@MaxGekk
Copy link
Member

MaxGekk commented May 6, 2023

+1, LGTM. Merging to master.
Thank you, @bozhang2820.

@MaxGekk MaxGekk closed this in 566980f May 6, 2023
LuciferYang pushed a commit to LuciferYang/spark that referenced this pull request May 10, 2023
### What changes were proposed in this pull request?
This change adds a error class INTERNAL_ERROR_EXECUTOR and change exceptions created in the executor package to use that error class.

### Why are the changes needed?
This is to move exceptions created in package org.apache.spark.executor to error class.

### Does this PR introduce _any_ user-facing change?
Yes. The exceptions created in package org.apache.spark.executor will change to be of error class INTERNAL_ERROR_EXECUTOR.

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

Closes apache#41048 from bozhang2820/spark-38462.

Authored-by: Bo Zhang <bo.zhang@databricks.com>
Signed-off-by: Max Gekk <max.gekk@gmail.com>
@bozhang2820 bozhang2820 deleted the spark-38462 branch May 23, 2023 05:33
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

Comments