-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Align java examples with scala ones #236
Conversation
Signed-off-by: Hongxin Liang <[email protected]>
import org.flyte.flytekit.jackson.JacksonSdkType; | ||
|
||
@AutoService(SdkWorkflow.class) | ||
public class SubWorkflow extends SdkWorkflow<SubWorkflow.Input, SubWorkflow.Output> { | ||
public class SubWorkflow extends SdkWorkflow<WelcomeWorkflow.Input, WelcomeWorkflow.Output> { |
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.
This becomes exactly the same as Scala.
import org.flyte.flytekit.jackson.JacksonSdkType; | ||
|
||
@AutoService(SdkWorkflow.class) | ||
public class SumWorkflow extends SdkWorkflow<SumWorkflow.Input, SumWorkflow.Output> { |
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.
This was previously SubWorkflow
in Java.
@@ -24,7 +24,7 @@ | |||
import org.flyte.flytekit.jackson.JacksonSdkType; | |||
|
|||
@AutoService(SdkWorkflow.class) | |||
public class UberWorkflow extends SdkWorkflow<UberWorkflow.Input, SubWorkflow.Output> { | |||
public class UberWorkflow extends SdkWorkflow<UberWorkflow.Input, SumWorkflow.Output> { |
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.
SubWorkflow
to SumWorkflow
change.
@@ -26,7 +26,7 @@ | |||
|
|||
/** Example workflow that takes a name and outputs a welcome message. */ | |||
@AutoService(SdkWorkflow.class) | |||
public class WelcomeWorkflow extends SdkWorkflow<WelcomeWorkflow.Input, AddQuestionTask.Output> { | |||
public class WelcomeWorkflow extends SdkWorkflow<WelcomeWorkflow.Input, WelcomeWorkflow.Output> { |
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.
To align with Scala implementation.
TL;DR
Align java examples with scala ones, for the subworkflow related things.
Type
Are all requirements met?
Complete description
It is a bit confusing that
SubWorkflow
in Java and Scala carries different meaning for different purpose. So we align the Java one with Scala.Tracking Issue
NA
Follow-up issue
NA