Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57118567
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -46,15 +48,24 @@ abstract class RuleExecutor[TreeType <: TreeNode[_]]
extends Logging {
/**
* An execution strategy for rules that indicates the maximum number of
executions. If the
* execution reaches fix point (i.e. converge) before maxIterations, it
will stop.
+ * If throwsExceptionUponMaxIterations is equal to true, it will issue
an exception
+ * TreeNodeException.
*/
- abstract class Strategy { def maxIterations: Int }
+ abstract class Strategy {
+ def maxIterations: Int
+ def throwsExceptionUponMaxIterations: Boolean
+ }
/** A strategy that only runs once. */
- case object Once extends Strategy { val maxIterations = 1 }
+ case object Once extends Strategy {
+ override val maxIterations = 1
+ override val throwsExceptionUponMaxIterations = false
+ }
/** A strategy that runs until fix point or maxIterations times,
whichever comes first. */
- case class FixedPoint(maxIterations: Int) extends Strategy
-
+ case class FixedPoint(maxIterations: Int) extends Strategy {
+ override val throwsExceptionUponMaxIterations: Boolean = if
(Utils.isTesting) true else false
--- End diff --
Just write `= Utils.isTesting`
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
