Skip to content

Conversation

@zhengruifeng
Copy link
Contributor

What changes were proposed in this pull request?

Refine error class MLAttributeNotAllowedException

Why are the changes needed?

this error message should contains the class name

Does this PR introduce any user-facing change?

yes, error change

How was this patch tested?

updated tests

Was this patch authored or co-authored using generative AI tooling?

no

zhengruifeng added a commit that referenced this pull request Feb 10, 2025
…xception`

### What changes were proposed in this pull request?
Refine error class `MLAttributeNotAllowedException`

### Why are the changes needed?
this error message should contains the class name

### Does this PR introduce _any_ user-facing change?
yes, error change

### How was this patch tested?
updated tests

### Was this patch authored or co-authored using generative AI tooling?
no

Closes #49860 from zhengruifeng/refine_attr_exception.

Authored-by: Ruifeng Zheng <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
(cherry picked from commit 34e6e44)
Signed-off-by: Ruifeng Zheng <[email protected]>
@zhengruifeng
Copy link
Contributor Author

thanks, merged to master/4.0

@zhengruifeng zhengruifeng deleted the refine_attr_exception branch February 10, 2025 04:11
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 14, 2025
…xception`

### What changes were proposed in this pull request?
Refine error class `MLAttributeNotAllowedException`

### Why are the changes needed?
this error message should contains the class name

### Does this PR introduce _any_ user-facing change?
yes, error change

### How was this patch tested?
updated tests

### Was this patch authored or co-authored using generative AI tooling?
no

Closes apache#49860 from zhengruifeng/refine_attr_exception.

Authored-by: Ruifeng Zheng <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
(cherry picked from commit c9a11e8)
Signed-off-by: Ruifeng Zheng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants