Skip to content

Conversation

@zhengruifeng
Copy link
Contributor

What changes were proposed in this pull request?

Support KolmogorovSmirnovTest on Connect

Why are the changes needed?

for feature parity

Does this PR introduce any user-facing change?

yes

How was this patch tested?

new test

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

no

Copy link
Contributor

@wbo4958 wbo4958 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

zhengruifeng added a commit that referenced this pull request Feb 5, 2025
… Connect

### What changes were proposed in this pull request?
Support `KolmogorovSmirnovTest` on Connect

### Why are the changes needed?
for feature parity

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

### How was this patch tested?
new test

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

Closes #49791 from zhengruifeng/ml_connect_kst.

Authored-by: Ruifeng Zheng <ruifengz@apache.org>
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
(cherry picked from commit 4138efd)
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
@zhengruifeng
Copy link
Contributor Author

thanks, merged to master/4.0

@zhengruifeng zhengruifeng deleted the ml_connect_kst branch February 5, 2025 03:56
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 14, 2025
… Connect

### What changes were proposed in this pull request?
Support `KolmogorovSmirnovTest` on Connect

### Why are the changes needed?
for feature parity

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

### How was this patch tested?
new test

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

Closes apache#49791 from zhengruifeng/ml_connect_kst.

Authored-by: Ruifeng Zheng <ruifengz@apache.org>
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
(cherry picked from commit fab8d0f)
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
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.

3 participants