Airflow: Add missing `labels` parameter to MLEngineTrainingOperator

Created on 5 Aug 2020  路  7Comments  路  Source: apache/airflow

Description

MLEngineTrainingOperator doesn't expose labels parameter, which is kind of surprising, considering that operators for BigQuery provide it (see e.g. here).

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.

good first issue feature Google

Most helpful comment

I can give a shot at picking this up.

All 7 comments

Thanks for opening your first issue here! Be sure to follow the issue template!

Hi. Would you like to prepare a PR to fix this problem? I will gladly support you if you have any questions.

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 馃檪

I can give a shot at picking this up.

@coopergillan I assigned you to this ticket.

@xor-xor - here ya go. Hoping this is what you had in mind!

https://github.com/apache/airflow/pull/10222

cc: @mik-laj @turbaszek

@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! 馃憤馃徎 馃檪

Was this page helpful?
0 / 5 - 0 ratings