carloea2 commented on code in PR #4024:
URL: https://github.com/apache/texera/pull/4024#discussion_r2512958136
##########
common/workflow-core/src/main/scala/org/apache/amber/core/tuple/Schema.scala:
##########
@@ -191,6 +191,10 @@ case class Schema @JsonCreator() (
object Schema {
+ /** Build a Schema with (name, type) pairs, in order, rejecting duplicates.
*/
+ def of(attrs: (String, AttributeType)*): Schema =
Review Comment:
The reason is to leverage them for future testing for the operators, I think
they are OK in this PR as adding them in an additional will add overhead and
complexity to a small utility method addition.
--
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]
