Skip to content

Commit 0f6dd70

Browse files
committed
docs: make comment more clear
-e Signed-off-by: machichima <[email protected]>
1 parent 6aae613 commit 0f6dd70

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

flytekit/core/array_node_map_task.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -380,8 +380,8 @@ def _raw_execute(self, **kwargs) -> Any:
380380
single_instance_inputs[k] = kwargs[k]
381381
try:
382382
o = self._run_task.execute(**single_instance_inputs)
383-
# For Container task, it will return the LiteralMap. We need to convert it to native
384-
# type here.
383+
# For running container task in local execution, it will return
384+
# the LiteralMap. We need to convert it to native type here.
385385
if isinstance(o, _literal_models.LiteralMap):
386386
vals = [Promise(var, o.literals[var]) for var in o.literals.keys()]
387387
result = create_task_output(vals, self.python_interface)

flytekit/tools/translator.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -166,10 +166,10 @@ def get_serializable_task(
166166
if settings.should_fast_serialize():
167167
# This handles container tasks.
168168
if container and isinstance(entity, (PythonAutoContainerTask, MapPythonTask, ArrayNodeMapTask)):
169-
# For fast registration, we'll need to muck with the command, but on
170-
# ly for certain kinds of tasks. Specifically,
171-
# tasks that rely on user code defined in the container. This should be encapsulated by the auto container
172-
# parent class
169+
# For fast registration, we'll need to muck with the command, but
170+
# only for certain kinds of tasks. Specifically, tasks that rely
171+
# on user code defined in the container. This should be
172+
# encapsulated by the auto container parent class
173173
container._args = prefix_with_fast_execute(settings, container.args or [])
174174

175175
# If the pod spec is not None, we have to get it again, because the one we retrieved above will be incorrect.

0 commit comments

Comments
 (0)