Skip to content

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