Skip to content

Commit cc0f412

Browse files
Do some rephrase in to_run.md (#2503)
1 parent 79dd80b commit cc0f412

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

doc/design/to_run.md

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -284,14 +284,13 @@ python /opt/sqlflow_run/python/ts_feature_extractor.py --time_column=t --value_c
284284

285285
### MaxCompute
286286

287-
Because MaxCompute requires that the Python program should be executed inside the
288-
PyODPS task node, we cannot execute the program in the step container just like
289-
above.
287+
For MaxCompute, we execute the Python program in the PyODPS task node instead of
288+
running it in the step container just like above.
290289

291290
- Use [goalisa](https://github.com/sql-machine-learning/goalisa) to submit a
292-
PyODPS task to MaxCompute in the way of sending a web request. The python
291+
PyODPS task to MaxCompute in the way of sending a web request. The Python
293292
program content and its arguments are the payload of the request.
294-
- Fetch the execution log and wait for the task done.
293+
- Fetch the execution logs and wait for the task done.
295294

296295
This step executes the following command in the step container:
297296

@@ -302,7 +301,7 @@ alisa.submitter /opt/sqlflow_run/python/ts_feature_extractor.py --time_column=t
302301
From the users' perspective, submitting PyODPS task using goalisa is the
303302
private protocol between SQLFlow and MaxCompute and expose too many details.
304303
And users only want to focus on the data processing logic in the Python
305-
program and don't want to pay attention to the execution different among
304+
program and don't want to pay attention to the execution difference among
306305
various platforms. To cover these details, we propose to add a module
307306
`sqlflow.runner` and set it as the entry point of the docker image for `TO RUN`.
308307
This module will get the platform type from the environment variable and decide

0 commit comments

Comments
 (0)