You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Change
Add support for tracking attributes through comprehensions.
Example var[2].field is declared unknown
for expression: var.exists(x, x.field == 'test')
I would expect that var[2].field might be returned as an unknown value.
Adding tracking here to keep feature parity with cel-cpp. I'm not sure what the current level of support is in cel-go. Let me know if I can add more detail.
The text was updated successfully, but these errors were encountered:
Thanks Jon, the details look fine. Support for unknowns currently stops at comprehensions, but is slightly different from what you might have seen before since the Go implementation optimizes the field selection behavior in addition to building an attribute trail.
It's totally possible to make the change you've suggested and it would have a dramatic speedup on comprehension processing since currently the code will stop evaluation at var when it could actually complete the evaluation. :) Thanks for the report.
Feature request checklist
Change
Add support for tracking attributes through comprehensions.
Example
var[2].field
is declared unknownfor expression:
var.exists(x, x.field == 'test')
I would expect that
var[2].field
might be returned as an unknown value.Adding tracking here to keep feature parity with cel-cpp. I'm not sure what the current level of support is in cel-go. Let me know if I can add more detail.
The text was updated successfully, but these errors were encountered: