[SPARK-7823] Refactor Strategy, Batch, FixedPoint out of class RuleExecutor[T]

[evacchi]

• Add companion object
• Add imports where necessary

[rxin]

Jenkins, test this please.

[SparkQA]

Test build #33354 has finished for PR 6349 at commit 8457feb.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • abstract class Strategy
  • case class Batch[TreeType <: TreeNode[_]](name: String, strategy: Strategy, rules: Rule[TreeType]*)
  • abstract class RuleExecutor[TreeType <: TreeNode[_]] extends Logging

[rxin]

Jenkins, retest this please.

[rxin]

Is this change identical to #6319 ?

[SparkQA]

Test build #33804 has finished for PR 6349 at commit b09fd68.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • abstract class Strategy
  • case class Batch[TreeType <: TreeNode[_]](
  • abstract class RuleExecutor[TreeType <: TreeNode[_]] extends Logging

[evacchi]

Not exactly, this version uses a companion object. Same purpose, though
Il 30/mag/2015 09:31, "Reynold Xin" notifications@github.com ha scritto:

Is this change identical to #6319
#6319 ?

—
Reply to this email directly or view it on GitHub
#6349 (comment).

[AmplabJenkins]

Can one of the admins verify this patch?

[marmbrus]

Hi @evacchi, thanks for working on this. Can we close this issue until you have time to bring it up to date? It would also be good to debate this vs #6319 on the JIRA.