acoustic-carpenter-78188
08/14/2023, 7:48 PMTaskExecutionContext is only used during maptask executions to perform bulk lookup for subtasks. The introduction of ArrayNode is mean to replace the maptask plugin and so when the k8s-array plugin is deprecated and remove we can remove the datacatalog client from the TaskExecutionContext.
Additionally, we can remove the PhaseWaitingForCache from Task phases, as caching is no longer performed at the task level.
What if we do not do this?
Upon removing the maptask k8s-array plugin the catalog client is no longer needed in the TaskExecutionContext. Cleaning this up is just being a good steward.
Related component(s)
No response
Are you sure this issue hasn't been raised already?
☑︎ Yes
Have you read the Code of Conduct?
☑︎ Yes
flyteorg/flyte