FrederikP commented on issue #3500: NIFI-6322: Reuse evaluator stored in 
CompiledExpression when creating a query.
URL: https://github.com/apache/nifi/pull/3500#issuecomment-496992773
 
 
   Hi @markap14 ,
   thanks for providing some context. I will try out some of those functions. I 
just assumed that, if the tests are green, it would be fine but I never 
actually checked that those functions are even covered.
   I might add some test cases then, because I think that might help in the 
future.
   
   And as a suggestion: If those functions are indeed problematic there could 
be a check to only rebuild evaluators with expressions that contain them. That 
would already help a ton. But before looking into this I'll see if it's a 
problem in the first place.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to