Remove the Base() *BaseExecutor
function from Executor
interface
#49490
Labels
type/enhancement
The issue or PR belongs to an enhancement.
Enhancement
To eliminate the usage of
Ctx
from the executors, as the first step, we can remove the abuse ofBase() *BaseExecutor
from theExecutor
interface. It can be replaced withActually, some helper functions can be removed, like
TryNewCacheChunk
,RetTypes
... They are just calling the methods of executors after this refractor.The executors can still use the
Ctx
inside, by calling.BaseExecutor.Ctx()
internally, as it can access theBaseExecutor
field and the type of the specific executor is clear.The text was updated successfully, but these errors were encountered: