OmerJog commented on a change in pull request #5419: [AIRFLOW-XXXX] Update
pydoc of mlengine_operator
URL: https://github.com/apache/airflow/pull/5419#discussion_r298353906
##########
File path: airflow/contrib/operators/mlengine_operator.py
##########
@@ -135,7 +135,7 @@ class MLEngineBatchPredictionOperator(BaseOperator):
:param max_worker_count: The maximum number of workers to be used
for parallel processing. Defaults to 10 if not specified.
- :type max_worker_count: int
+ :type max_worker_count: string
Review comment:
@dossett The 2nd option suggested is supported and we have places where we
already use it like:
https://github.com/apache/airflow/blob/master/airflow/contrib/operators/bigquery_operator.py#L51
so I say go for this. Maybe even extend the explanation a bit so users will
understand it better,
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
