rakesh-rsky commented on PR #11296: URL: https://github.com/apache/nifi/pull/11296#issuecomment-4605551215
@pvillard31 you are right I have updated expression-language-guide.adoc for both plusDuration and minusDuration to document the new optional timezone argument, added a DST example row to each example table, and added a DST boundary scenario to each Calendar-Aware Behavior table. Also added fail-fast validation for literal timezone arguments in the constructor of AbstractDateArithmeticEvaluator, mirroring the existing pattern used for the amount literal via DateAmountParser.validate. For dynamic (non-literal) timezone expressions, the ZoneId.of() call in evaluate() is now also wrapped to throw AttributeExpressionLanguageException consistently. Added test coverage for both cases in TestQuery. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
