Not in prod, so we can sign this off directly.

This is supplanted by the composition v2 work.

In T365515#10341215, @Jdforrester-WMF wrote:

Is this being over-taken by the Rust work? I assume so, but let's leave this open until confirmed.

1. Since the limits are under our contro, we just shouldn't set them very low 😄 ?
2. This will no longer be relevant when we switch to Rust.

This no longer makes sense to do; we will instead proceed with the implementation described in T413943.

After a recent refactor, this no longer makes sense to do.

We are now doing this, as much as possible.

I don't foresee us wanting this functionality; we should only accept a single format now.

I think this is not going to be a problem anymore, especially with compositions V2. Let's close it.

The plot thickens.

I'm still able to reproduce this on prod. When I run in a local stack, the same requests work perfectly. Unfortunately, our logs don't capture the relevant stack traces (or even tell us where the errors are coming from). We also don't log the requests made to the evaluator. If we logged that, we could at least tell whether this issue was due to 1) an invalid payload constructed by the orchestrator or 2) differences in how prod vs. local evaluators process the same payload.

Moved directly fo "Needs Sign-off" because this is a test-only change.

Ah, I see. I think we will be phasing that kind of validation out, so it is not (or will not be, in the near future) by design that a type would need to be validated.

This works! We can run functions like Z883 (and Z882, Z881) with no issues. Recommend resolving.

In T371835#11496583, @DMartin-WMF wrote:

At this point in time, it's not clear whether/when we will implement asynchronous reentrancy; even if we do, we'll need something like a tagging mechanism, not mutexes.

BUILTIN_ABSTRACT_: migrate
BUILTIN_ADD_DEBUG_LOG_: WAIT; this one will be tricky
BUILTIN_CHARS_TO_STRING_: migrate
BUILTIN_CONS_: migrate
BUILTIN_ECHO_: migrate
BUILTIN_EMPTY_: IGNORE; this shouldn't be needed
BUILTIN_EMPTY_VALIDATOR_: IGNORE; this shouldn't be needed
BUILTIN_EQUALS_BOOLEAN_: migrate
BUILTIN_EQUALS_HTML_FRAGMENT_: migrate
BUILTIN_EQUALS_LIST_: migrate
BUILTIN_EQUALS_STRING_: migrate
BUILTIN_EQUALS_WIKIDATA_CLAIM_: migrate
BUILTIN_EQUALS_WIKIDATA_ENUM_: migrate
BUILTIN_EQUALS_WIKIDATA_ITEM_: migrate
BUILTIN_EQUALS_WIKIDATA_LEXEME: migrate
BUILTIN_EQUALS_WIKIDATA_LEXEME_FORM: migrate
BUILTIN_EQUALS_WIKIDATA_LEXEME_SENSE: migrate
BUILTIN_EQUALS_WIKIDATA_PROPERTY_: migrate
BUILTIN_EQUALS_WIKIDATA_STATEMENT_: migrate
BUILTIN_ERRORTYPE_TO_TYPE_: migrate
BUILTIN_ERROR_TYPE_VALIDATOR_: IGNORE; this shouldn't be needed
BUILTIN_FETCH_PERSISTENT_ZOBJECT_: migrate
BUILTIN_FETCH_WIKIDATA_ENTITIES_: migrate
BUILTIN_FETCH_WIKIDATA_ITEM_: migrate
BUILTIN_FETCH_WIKIDATA_LEXEME_: migrate
BUILTIN_FETCH_WIKIDATA_LEXEME_FORM_: migrate
BUILTIN_FETCH_WIKIDATA_LEXEME_SENSE_: migrate
BUILTIN_FETCH_WIKIDATA_PROPERTY_: migrate
BUILTIN_FILTER_FUNCTION_: migrate
BUILTIN_FIND_LEXEMES_FOR_ITEM_: migrate
BUILTIN_FIND_LEXEMES_FOR_LEXEME_SENSE_: migrate
BUILTIN_FIRST_: migrate
BUILTIN_FUNCTION_CALL_VALIDATOR_: IGNORE; this shouldn't be needed
BUILTIN_FUNCTION_CALL_VALIDATOR_INTERNAL_: IGNORE; this shouldn't be needed
BUILTIN_GENERIC_LIST_TYPE_: WAIT; this one will be tricky
BUILTIN_GENERIC_MAP_TYPE_: WAIT; this one will be tricky
BUILTIN_GENERIC_PAIR_TYPE_: WAIT; this one will be tricky
BUILTIN_GENERIC_WIKIDATA_ENUM_TYPE_: WAIT; this one will be tricky
BUILTIN_GET_ENVELOPE_: WAIT; this one will be tricky
BUILTIN_GET_ERROR_: WAIT; this one will be tricky
BUILTIN_HEAD_: migrate
BUILTIN_IF_: migrate
BUILTIN_IS_ERROR_TYPE_: migrate
BUILTIN_LANGUAGE_CODE_TO_LANGUAGE_: migrate
BUILTIN_MAP_FUNCTION_: migrate
BUILTIN_MULTILINGUAL_STRINGSET_VALIDATOR_: IGNORE; this shouldn't be needed
BUILTIN_MULTILINGUAL_TEXT_VALIDATOR_: IGNORE; this shouldn't be needed
BUILTIN_REDUCE_FUNCTION_: migrate
BUILTIN_REF_FROM_WIKIDATA_ENUM_: migrate
BUILTIN_REIFY_: migrate
BUILTIN_SAME_: migrate
BUILTIN_SCHEMA_VALIDATOR_: IGNORE; this shouldn't be needed
BUILTIN_SECOND_: migrate
BUILTIN_STRING_TO_CHARS_: migrate
BUILTIN_TAIL_: migrate
BUILTIN_THROW_: migrate
BUILTIN_TRIGGER_METADATA_: WAIT; this one will be tricky
BUILTIN_TRY_CATCH_: WAIT; this one will be tricky
BUILTIN_UNQUOTE_: migrate
BUILTIN_VALUE_BY_KEY_: migrate
BUILTIN_VALUES_BY_KEYS_: migrate
BUILTIN_VALUES_FROM_WIKIDATA_ENUM_: migrate
BUILTIN_WIKIDATA_ENUM_VALIDATOR_: IGNORE; this shouldn't be needed
BUILTIN_WRAP_MONOLINGUAL_TEXT_: migrate

This won't truly be deployed for a very long time, so I'm closing it out directly.