feel-scala icon indicating copy to clipboard operation
feel-scala copied to clipboard

Functions `put all` and `context` do not propagate errors to the result

Open pihme opened this issue 2 years ago • 1 comments

Describe the bug Currently the built-in functions put all and context do not propagate errors that are passed in as argument. Instead, they return null in these cases and the error gets lost.

This is the root cause for: https://github.com/camunda/zeebe/issues/9543

To Reproduce see ticket linked above

Expected behavior

  • If an error is passed in, this error should be returned as the result of the function
  • If multiple errors are passed in, the first error should be returned as the result of the function

Environment

  • FEEL engine version: master
  • Affects:
    • Zeebe broker: https://github.com/camunda/zeebe/issues/9543

pihme avatar Jun 30 '22 12:06 pihme

Implementation was started in branch 446-propagate...

It is currently blocked by

  • #260

pihme avatar Jul 01 '22 09:07 pihme

We will not work on this issue. Instead, we will look at the topic when implementing custom error handling. The goal would be that the function returns null and reports an error.

saig0 avatar Jun 01 '23 08:06 saig0