-
Notifications
You must be signed in to change notification settings - Fork 53
Support BQ plugin to write structured dataset as output #233
Conversation
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #233 +/- ##
==========================================
+ Coverage 62.29% 62.41% +0.11%
==========================================
Files 141 142 +1
Lines 8811 8831 +20
==========================================
+ Hits 5489 5512 +23
+ Misses 2823 2819 -4
- Partials 499 500 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -273,13 +280,54 @@ func (p Plugin) Status(_ context.Context, tCtx webapi.StatusContext) (phase core | |||
resource.Status.ErrorResult.Message, | |||
taskInfo), nil | |||
} | |||
|
|||
err = writeOutput(ctx, tCtx, resource.OutputLocation) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm confused, were we not writing an output before?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We didn't write any output before.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 but a question
* BQ output structured dataset Signed-off-by: Kevin Su <[email protected]> * Fixed test Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su [email protected]
TL;DR
support BQ plugin to write structured dataset as output
Type
Are all requirements met?
Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
Related to flyteorg/flytekit#789
Follow-up issue
NA