You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/08/05 07:08:44 UTC
[GitHub] [airflow] xor-xor opened a new issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
xor-xor opened a new issue #10165:
URL: https://github.com/apache/airflow/issues/10165
**Description**
`MLEngineTrainingOperator` doesn't expose `labels` parameter, which is kind of surprising, considering that operators for BigQuery provide it (see e.g. [here](https://airflow.apache.org/docs/stable/_api/airflow/contrib/operators/bigquery_operator/index.html#airflow.contrib.operators.bigquery_operator.BigQueryOperator)).
BTW, this also applies to `MLEngineBatchPredictionOperator`.
**Use case / motivation**
Accounting, mostly - with a considerable number of training/prediction jobs launched daily on ML Engine / AI Platform, labels become crucial for this task.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] boring-cyborg[bot] commented on issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #10165:
URL: https://github.com/apache/airflow/issues/10165#issuecomment-669022131
Thanks for opening your first issue here! Be sure to follow the issue template!
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] mik-laj commented on issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
mik-laj commented on issue #10165:
URL: https://github.com/apache/airflow/issues/10165#issuecomment-670084435
@coopergillan I assigned you to this ticket.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] mik-laj commented on issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
mik-laj commented on issue #10165:
URL: https://github.com/apache/airflow/issues/10165#issuecomment-669024031
Hi. Would you like to prepare a PR to fix this problem? I will gladly support you if you have any questions.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] coopergillan commented on issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
coopergillan commented on issue #10165:
URL: https://github.com/apache/airflow/issues/10165#issuecomment-670037712
I can give a shot at picking this up.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] coopergillan commented on issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
coopergillan commented on issue #10165:
URL: https://github.com/apache/airflow/issues/10165#issuecomment-670561184
@xor-xor - here ya go. Hoping this is what you had in mind!
https://github.com/apache/airflow/pull/10222
cc: @mik-laj @turbaszek
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] xor-xor commented on issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
xor-xor commented on issue #10165:
URL: https://github.com/apache/airflow/issues/10165#issuecomment-669075466
> Would you like to prepare a PR to fix this problem?
Hard to say... Seems to be an easy thing to implement, but I'm a bit short on time recently. We will see 🙂
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] mik-laj closed issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
mik-laj closed issue #10165:
URL: https://github.com/apache/airflow/issues/10165
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] xor-xor commented on issue #10165: Add missing `labels` parameter to MLEngineTrainingOperator
Posted by GitBox <gi...@apache.org>.
xor-xor commented on issue #10165:
URL: https://github.com/apache/airflow/issues/10165#issuecomment-671566348
> @xor-xor - here ya go. Hoping this is what you had in mind!
Yep, exactly what I was looking for before I opened this issue. Thank you! 👍🏻 🙂
----------------------------------------------------------------
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:
users@infra.apache.org