Join Slack
Powered by
<#180 Remove duplication of workflow inputs and ou...
# flyte-github
a
acoustic-carpenter-78188
01/31/2023, 4:08 PM
#180 Remove duplication of workflow inputs and outputs #minor
Pull request opened by
sonjaer
TL;DR
Currently, users need to define outputs and inputs twice through SdkWorkflow<InputT, OutputT> and in builder.inputOf() and builder.output() which forces users to specify duplicate information which is error prone. This PR removes the ability to specify inputs/outputs through the builder. Tested here: •
http://flyteconsole.services.gew1.spotify.net/console/projects/sonjae-test-flyte-scio/domains/development/executions/f64d4bff0192d4ceb84a?duration=all&fromExecutionNav=true
•
http://flyteconsole.services.gew1.spotify.net/console/projects/sonjae-test-flyte-scio/domains/development/executions/f94605359345d47ca901?duration=all&fromExecutionNav=true
•
http://flyteconsole.services.gew1.spotify.net/console/projects/sonjae-test-flyte-scio/domains/development/executions/fcb73fe3a3ef94dbd871?duration=all&fromExecutionNav=true
Type
☐ Bug Fix ☑︎ Feature ☐ Plugin
Are all requirements met?
☑︎ Code completed ☐ Smoke tested ☑︎ Unit tests added ☑︎ Code documentation added ☐ Any pending items have an associated Issue
Complete description
See
flyteorg/flyte#3303
for more information
Tracking Issue
Work towards
flyteorg/flyte#3303
Follow-up issue
NA
flyteorg/flytekit-java
✅ All checks have passed 3/3 successful checks
acoustic-carpenter-78188
01/31/2023, 4:08 PM
#180 Remove duplication of workflow inputs and outputs #minor
Pull request ready for review by
sonjaer
flyteorg/flytekit-java
acoustic-carpenter-78188
02/01/2023, 7:37 AM
#180 Remove duplication of workflow inputs and outputs #minor
Pull request merged by
sonjaer
flyteorg/flytekit-java
Open in Slack
Previous
Next