Page MenuHomePhabricator

Revisit how implementationId is captured for metadata map
Open, LowPublic

Description

Description

Desired behavior/Acceptance criteria (returned value, expected error, performance expectations, etc.)

  • The current behavior is correct (*), but the implementation can be made nicer if and when T320557 is completed.

(*) The current implementation will catch persistent IDs for Composition and Evaluated implementations, if they show up in Z8K4, in executeInternal(), as a Z9. It will catch persistent IDs for Built-in implementations if they show up (in the same place) as Z14K4.Z6K1. This is probably okay for now, but T320557 might allow us to cover more possible cases in a more principled and neater manner.