When iterator of RuleExecutor breaks, the num of iterator should be iteration - 1 not iteration.

pzzs · pzzs · commit 46514b6a3cf4 · 2014-11-10T12:16:54.000+08:00
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
@@ -79,7 +79,7 @@ abstract class RuleExecutor[TreeType <: TreeNode[_]] extends Logging {
         }
 
         if (curPlan.fastEquals(lastPlan)) {
-          logTrace(s"Fixed point reached for batch ${batch.name} after $iteration iterations.")
+          logTrace(s"Fixed point reached for batch ${batch.name} after ${iteration - 1} iterations.")
           continue = false
         }
         lastPlan = curPlan

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ abstract class RuleExecutor[TreeType <: TreeNode[_]] extends Logging {`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`if (curPlan.fastEquals(lastPlan)) {`
`82`		`- logTrace(s"Fixed point reached for batch ${batch.name} after $iteration iterations.")`
	`82`	`+ logTrace(s"Fixed point reached for batch ${batch.name} after ${iteration - 1} iterations.")`
`83`	`83`	`continue = false`
`84`	`84`	`}`
`85`	`85`	`lastPlan = curPlan`