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]


Reply via email to