Skip to content

[SPARK-34106][ML] Hide FValueTest and AnovaTest#31176

Closed
zhengruifeng wants to merge 2 commits intoapache:masterfrom
zhengruifeng:hide_fvalue_anova_test
Closed

[SPARK-34106][ML] Hide FValueTest and AnovaTest#31176
zhengruifeng wants to merge 2 commits intoapache:masterfrom
zhengruifeng:hide_fvalue_anova_test

Conversation

@zhengruifeng
Copy link
Contributor

What changes were proposed in this pull request?

Hide FValueTest and AnovaTest:
1, make them private[spark];
2, remove related doc;
3, remove them in the py side

Why are the changes needed?

they are not very practical for big data. If there are valid use cases, we should see more requests from the community.

Does this PR introduce any user-facing change?

No, these tests had not been released.

How was this patch tested?

existing testsuites

@SparkQA
Copy link

SparkQA commented Jan 14, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38620/

@SparkQA
Copy link

SparkQA commented Jan 14, 2021

Test build #134033 has finished for PR 31176 at commit 437cf03.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 14, 2021

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38620/

@SparkQA
Copy link

SparkQA commented Jan 14, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38624/

@SparkQA
Copy link

SparkQA commented Jan 14, 2021

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38624/

@SparkQA
Copy link

SparkQA commented Jan 14, 2021

Test build #134037 has finished for PR 31176 at commit 4bdc698.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zhengruifeng zhengruifeng deleted the hide_fvalue_anova_test branch January 14, 2021 05:55
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