Skip to content

Conversation

@cloud-fan
Copy link
Contributor

a follow up of #7479.
The TreeNode is the root case of the requirement of self: Product => stuff, so why not make TreeNode extend Product?

@SparkQA
Copy link

SparkQA commented Jul 18, 2015

Test build #37715 has finished for PR 7495 at commit 8676af7.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • abstract class Expression extends TreeNode[Expression]
    • abstract class LogicalPlan extends QueryPlan[LogicalPlan] with Logging
    • abstract class TreeNode[BaseType <: TreeNode[BaseType]] extends Product
    • abstract class SparkPlan extends QueryPlan[SparkPlan] with Logging with Serializable

@rxin
Copy link
Contributor

rxin commented Jul 18, 2015

Thanks - merging this in.

@asfgit asfgit closed this in 1b4ff05 Jul 18, 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.

3 participants