Skip to content

[SPARK-9987][SQL]Implement the local Aggregate operator #8769

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

Closed
wants to merge 3 commits into from
Closed

[SPARK-9987][SQL]Implement the local Aggregate operator #8769

wants to merge 3 commits into from

Conversation

zsxwing
Copy link
Member

@zsxwing zsxwing commented Sep 15, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Sep 15, 2015

Test build #42490 has finished for PR 8769 at commit 640a74e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class AggregateNode(

@SparkQA
Copy link

SparkQA commented Sep 16, 2015

Test build #42518 has finished for PR 8769 at commit 1e5ed75.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class TaskCommitDenied(
    • final val probabilityCol: Param[String] = new Param[String](this, "probabilityCol", "Column name for predicted class conditional probabilities. Note: Not all models output well-calibrated probability estimates! These probabilities should be treated as confidences, not precise probabilities")
    • case class AggregateNode(
    • abstract class LocalNode(conf: SQLConf) extends QueryPlan[LocalNode] with Logging

@rxin
Copy link
Contributor

rxin commented Sep 20, 2015

Actually can you close this one? I was thinking it'd make more sense to only keep the Tungsten version here ...

(You shouldn't be doing this - will get somebody else to pick this up and use your code. Thanks!)

@andrewor14
Copy link
Contributor

@zsxwing let's close this since we're removing the old aggregate operator.

@zsxwing zsxwing closed this Nov 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants