jadams-tresys commented on a change in pull request #450:
URL: https://github.com/apache/incubator-daffodil/pull/450#discussion_r538793602
##########
File path:
daffodil-runtime1/src/main/scala/org/apache/daffodil/processors/VariableMap1.scala
##########
@@ -128,6 +184,10 @@ class VariableHasNoValue(qname: NamedQName, context:
VariableRuntimeData) extend
"Variable map (runtime): variable %s has no value. It was not set, and has
no default value.".format(qname))
with RetryableException
+class VariableCircularDefinition(qname: NamedQName, context:
VariableRuntimeData) extends VariableException(qname, context,
+ "Variable map (runtime): variable %s is part of a circular definition with
other variables".format(qname))
+ with RetryableException
Review comment:
I think I had retryable here simply because I copy and pasted from the
exception above it. Will add scaladoc
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]