Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "[SPARK-17704][ML][MLLIB] ChiSqSelector performance improvement." #15298

Closed
wants to merge 1 commit into from

Conversation

yanboliang
Copy link
Contributor

What changes were proposed in this pull request?

Revert "[SPARK-17704][ML][MLLIB] ChiSqSelector performance improvement."
This reverts commit f7082ac.

How was this patch tested?

@srowen
Copy link
Member

srowen commented Sep 29, 2016

Yes that's fine, note you can also just revert the PR and push that commit straight to the apache git remote if you want to.

@yanboliang
Copy link
Contributor Author

@srowen Sure. Thanks.

@srowen
Copy link
Member

srowen commented Sep 29, 2016

OK instead of reverting this, maybe it's easier to just consider an additional change. What do you think of #15299?

@yanboliang
Copy link
Contributor Author

OK, I'll close this one and move to #15299 . Thanks.

@yanboliang yanboliang closed this Sep 29, 2016
@yanboliang yanboliang deleted the revert-17704 branch September 29, 2016 13:52
@SparkQA
Copy link

SparkQA commented Sep 29, 2016

Test build #66098 has finished for PR 15298 at commit e034f8b.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants