Github user cestella commented on the issue:

    https://github.com/apache/metron/pull/687
  
    @nickwallen Yes, you are absolutely correct.  Right now we do not have an 
approach to take a list of stellar expressions that are assignments, a variable 
resolver, a function resolver and execute that list of statements in a way that 
variables are updated.  At the time we punted on it due to limitations in the 
VariableResolver to update (also, it's not clear how to do that generally or 
even specifically in all circumstances).  All that being said, it might be nice 
to add a method to do that in `StellarAssignment` that presumes variable 
containers are maps.
    
    Anyway, that aside, this PR seems to not *force* that change since it's 
really just moving variable assignment into the language and not updating the 
resolver explicitly.  I'd suggest we tackle that as a follow-on and make a 
stellar-generic approach to update variable resolvers with new variables.


---

Reply via email to