While we were transitioning from the asynchronous implementation of the runtime to the new synchronous implementation of the runtime, the ExecutionRuntimeConfig allowed selecting which implementation should be used. Now there's only one, so the enum is no longer useful. However, the type will be used to pass additional configuration options in the future, so there's a reason to still keep it.
Proposal
Change the ExecutionRuntimeConfig into a struct, which for now is empty.
Test Plan
This is an internal refactor, so existing tests should catch any regressions.
Release Plan
Nothing is needed because this is an internal refactor.
Motivation
While we were transitioning from the asynchronous implementation of the runtime to the new synchronous implementation of the runtime, the
ExecutionRuntimeConfig
allowed selecting which implementation should be used. Now there's only one, so theenum
is no longer useful. However, the type will be used to pass additional configuration options in the future, so there's a reason to still keep it.Proposal
Change the
ExecutionRuntimeConfig
into astruct
, which for now is empty.Test Plan
This is an internal refactor, so existing tests should catch any regressions.
Release Plan
Nothing is needed because this is an internal refactor.
Links