kaxil commented on code in PR #76019:
URL: https://github.com/apache/airflow/pull/76019#discussion_r4437467950
##########
java-sdk/example/src/java/org/apache/airflow/example/InterfaceExampleBuilder.java:
##########
@@ -88,11 +88,4 @@ public void execute(@NotNull Context context, Client client,
SummarizeInput inpu
}
}
}
-
- public static DagDef build() {
Review Comment:
Nit: with `build()` gone this class only holds the three task classes, so
the `Builder` suffix reads a bit odd in an example people copy. Renaming it to
`InterfaceExample` would line up with `AnnotationExample` and mirror
`nativedag.InterfaceExample`. Happy for that to be a follow-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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
