fix: operators non-blank attribute validation (#4006)

<!--
Thanks for sending a pull request (PR)! Here are some tips for you:
1. If this is your first time, please read our contributor guidelines:
[Contributing to
Texera](https://github.com/apache/texera/blob/main/CONTRIBUTING.md)
  2. Ensure you have added or run the appropriate tests for your PR
  3. If the PR is work in progress, mark it a draft on GitHub.
  4. Please write your PR title to summarize what this PR proposes, we 
    are following Conventional Commits style for PR titles as well.
  5. Be sure to keep the PR description updated to reflect all changes.
-->

### What changes were proposed in this PR?
<!--
Please clarify what changes you are proposing. The purpose of this
section
is to outline the changes. Here are some tips for you:
  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, you can clarify why it is a bug.
  3. If it is a refactoring, clarify what has been changed.
  3. It would be helpful to include a before-and-after comparison using 
     screenshots or GIFs.
  4. Please consider writing useful notes for better and faster reviews.
-->
This PR includes two changes:
- introduces `javax.validation:validation-api:2.0.1.Final` to the build
configuration of the `workflow-core` package, as annotations are needed
from this dependency. Some other packages already have this, e.g.,
`workflow-operator` and `amber`.
- adds non-blank / non-null attribute validation annotation to basic
chart operators.

### Any related issues, documentation, discussions?
<!--
Please use this section to link other resources if not mentioned
already.
1. If this PR fixes an issue, please include `Fixes #1234`, `Resolves
#1234`
or `Closes #1234`. If it is only related, simply mention the issue
number.
  2. If there is design documentation, please add the link.
  3. If there is a discussion in the mailing list, please add the link.
-->
Fixes #3692

Related to: #3555, #3656, #3192, #3914

### How was this PR tested?
<!--
If tests were added, say they were added here. Or simply mention that if
the PR
is tested with existing test cases. Make sure to include/update test
cases that
check the changes thoroughly including negative and positive cases if
possible.
If it was tested in a way different from regular unit tests, please
clarify how
you tested step by step, ideally copy and paste-able, so that other
reviewers can
test and check, and descendants can verify in the future. If tests were
not added,
please describe why they were not added and/or why it was difficult to
add.
-->
Each operator is added to a testing workflow to check if the validation
will pass when those attributes are empty.


### Was this PR authored or co-authored using generative AI tooling?
<!--
If generative AI tooling has been used in the process of authoring this
PR,
please include the phrase: 'Generated-by: ' followed by the name of the
tool
and its version. If no, write 'No'. 
Please refer to the [ASF Generative Tooling
Guidance](https://www.apache.org/legal/generative-tooling.html) for
details.
-->
GitHub Copilot helped with coding those annotations.

---------

Co-authored-by: Xiaozhen Liu <[email protected]>
Co-authored-by: Chen Li <[email protected]>

Author: 3 people

common/workflow-core/build.sbt

@@ -97,6 +97,7 @@ libraryDependencies ++= Seq(
 
 val jacksonVersion = "2.15.1"
 libraryDependencies ++= Seq(
+  "javax.validation" % "validation-api" % "2.0.1.Final",
   "com.fasterxml.jackson.core" % "jackson-databind" % jacksonVersion,        // Jackson Databind
   "com.fasterxml.jackson.module" % "jackson-module-kotlin" % jacksonVersion % Test,   // Jackson Kotlin Module
   "com.fasterxml.jackson.datatype" % "jackson-datatype-jdk8" % jacksonVersion % Test, // Jackson JDK8 Datatypes

common/workflow-core/src/main/scala/org/apache/amber/core/tuple/Attribute.java

@@ -22,6 +22,7 @@
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
 
+import javax.validation.constraints.NotBlank;
 import javax.validation.constraints.NotNull;
 import java.io.Serializable;
 
@@ -47,7 +48,7 @@ public Attribute(
     }
 
     @JsonProperty(value = "attributeName", required = true)
-    @NotNull(message = "Attribute name is required")
+    @NotBlank(message = "Attribute name is required")
     public String getName() {
         return attributeName;
     }

common/workflow-operator/src/main/scala/org/apache/amber/operator/timeSeriesPlot/TimeSeriesPlot.scala

@@ -27,19 +27,23 @@ import org.apache.amber.operator.PythonOperatorDescriptor
 import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 import org.apache.amber.operator.metadata.{OperatorGroupConstants, OperatorInfo}
 
+import javax.validation.constraints.{NotBlank, NotNull}
+
 class TimeSeriesOpDesc extends PythonOperatorDescriptor {
 
   @JsonProperty(value = "timeColumn", required = true)
   @JsonSchemaTitle("Time Column")
   @JsonPropertyDescription("The column containing time/date values (e.g., Date, Timestamp).")
   @AutofillAttributeName
+  @NotNull(message = "Time Column cannot be empty")
   var timeColumn: String = ""
 
   @JsonProperty(value = "valueColumn", required = true)
   @JsonSchemaTitle("Value Column")
   @JsonPropertyDescription("The numerical column to plot on the Y-axis (e.g., Sales, Temperature).")
   @JsonSchemaInject(json = """{"enum": "autofill"}""")
   @AutofillAttributeName
+  @NotNull(message = "Value Column cannot be empty")
   var valueColumn: String = ""
 
   @JsonProperty(value = "categoryColumn", required = false, defaultValue = "No Selection")
@@ -57,6 +61,7 @@ class TimeSeriesOpDesc extends PythonOperatorDescriptor {
   @JsonProperty(value = "line", defaultValue = "line", required = true)
   @JsonSchemaTitle("Plot Type")
   @JsonPropertyDescription("Select the type of time series plot (line, area).")
+  @NotBlank(message = "Plot Type cannot be empty")
   var plotType: String = "line"
 
   @JsonProperty(value = "slider", defaultValue = "false")

common/workflow-operator/src/main/scala/org/apache/amber/operator/visualization/DotPlot/DotPlotOpDesc.scala

@@ -28,12 +28,15 @@ import org.apache.amber.operator.PythonOperatorDescriptor
 import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 import org.apache.amber.operator.metadata.{OperatorGroupConstants, OperatorInfo}
 
+import javax.validation.constraints.NotNull
+
 class DotPlotOpDesc extends PythonOperatorDescriptor {
 
   @JsonProperty(value = "Count Attribute", required = true)
   @JsonSchemaTitle("Count Attribute")
   @JsonPropertyDescription("the attribute for the counting of the dot plot")
   @AutofillAttributeName
+  @NotNull(message = "Count Attribute column cannot be empty")
   var countAttribute: String = ""
 
   override def getOutputSchemas(

common/workflow-operator/src/main/scala/org/apache/amber/operator/visualization/IcicleChart/IcicleChartOpDesc.scala

@@ -29,6 +29,8 @@ import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 import org.apache.amber.operator.metadata.{OperatorGroupConstants, OperatorInfo}
 import org.apache.amber.operator.visualization.hierarchychart.HierarchySection
 
+import javax.validation.constraints.{NotEmpty, NotNull}
+
 // type constraint: value can only be numeric
 @JsonSchemaInject(json = """
 {
@@ -45,12 +47,14 @@ class IcicleChartOpDesc extends PythonOperatorDescriptor {
   @JsonPropertyDescription(
     "hierarchy of attributes from a root (higher-level category) to leaves (lower-level category)"
   )
+  @NotEmpty(message = "Hierarchy path list cannot be empty")
   var hierarchy: List[HierarchySection] = List()
 
   @JsonProperty(value = "value", required = true)
   @JsonSchemaTitle("Value Column")
   @JsonPropertyDescription("the value associated with the size of each sector in the chart")
   @AutofillAttributeName
+  @NotNull(message = "Value column cannot be empty")
   var value: String = ""
 
   override def getOutputSchemas(

common/workflow-operator/src/main/scala/org/apache/amber/operator/visualization/barChart/BarChartOpDesc.scala

@@ -28,6 +28,8 @@ import org.apache.amber.operator.PythonOperatorDescriptor
 import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 import org.apache.amber.operator.metadata.{OperatorGroupConstants, OperatorInfo}
 
+import javax.validation.constraints.NotNull
+
 //type constraint: value can only be numeric
 @JsonSchemaInject(json = """
 {
@@ -44,12 +46,14 @@ class BarChartOpDesc extends PythonOperatorDescriptor {
   @JsonSchemaTitle("Value Column")
   @JsonPropertyDescription("The value associated with each category")
   @AutofillAttributeName
+  @NotNull(message = "Value column cannot be empty")
   var value: String = ""
 
   @JsonProperty(required = true)
   @JsonSchemaTitle("Fields")
   @JsonPropertyDescription("Visualize categorical data in a Bar Chart")
   @AutofillAttributeName
+  @NotNull(message = "Fields cannot be empty")
   var fields: String = ""
 
   @JsonProperty(defaultValue = "No Selection", required = false)
@@ -88,8 +92,8 @@ class BarChartOpDesc extends PythonOperatorDescriptor {
     )
 
   def manipulateTable(): String = {
-    assert(value.nonEmpty)
-    assert(fields.nonEmpty)
+    assert(value.nonEmpty, "Value column cannot be empty")
+    assert(fields.nonEmpty, "Fields cannot be empty")
     s"""
        |        table = table.dropna(subset = ['$value', '$fields']) #remove missing values
        |""".stripMargin

common/workflow-operator/src/main/scala/org/apache/amber/operator/visualization/bubbleChart/BubbleChartOpDesc.scala

@@ -28,6 +28,8 @@ import org.apache.amber.operator.PythonOperatorDescriptor
 import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 import org.apache.amber.operator.metadata.{OperatorGroupConstants, OperatorInfo}
 
+import javax.validation.constraints.NotNull
+
 /**
   * Visualization Operator to visualize results as a Bubble Chart
   * User specifies 2 columns to use for the x, y labels. Size of bubbles determined via
@@ -40,17 +42,23 @@ class BubbleChartOpDesc extends PythonOperatorDescriptor {
   @JsonProperty(value = "xValue", required = true)
   @JsonSchemaTitle("X-Column")
   @JsonPropertyDescription("Data column for the x-axis")
-  @AutofillAttributeName var xValue: String = ""
+  @AutofillAttributeName
+  @NotNull(message = "xValue column cannot be empty")
+  var xValue: String = ""
 
   @JsonProperty(value = "yValue", required = true)
   @JsonSchemaTitle("Y-Column")
   @JsonPropertyDescription("Data column for the y-axis")
-  @AutofillAttributeName var yValue: String = ""
+  @AutofillAttributeName
+  @NotNull(message = "yValue column cannot be empty")
+  var yValue: String = ""
 
   @JsonProperty(value = "zValue", required = true)
   @JsonSchemaTitle("Z-Column")
   @JsonPropertyDescription("Data column to determine bubble size")
-  @AutofillAttributeName var zValue: String = ""
+  @AutofillAttributeName
+  @NotNull(message = "zValue column cannot be empty")
+  var zValue: String = ""
 
   @JsonProperty(value = "enableColor", defaultValue = "false")
   @JsonSchemaTitle("Enable Color")
@@ -60,7 +68,9 @@ class BubbleChartOpDesc extends PythonOperatorDescriptor {
   @JsonProperty(value = "colorCategory", required = true)
   @JsonSchemaTitle("Color-Column")
   @JsonPropertyDescription("Picks data column to color bubbles with if color is enabled")
-  @AutofillAttributeName var colorCategory: String = ""
+  @AutofillAttributeName
+  @NotNull(message = "colorCategory column cannot be empty")
+  var colorCategory: String = ""
 
   override def getOutputSchemas(
       inputSchemas: Map[PortIdentity, Schema]

common/workflow-operator/src/main/scala/org/apache/amber/operator/visualization/dumbbellPlot/DumbbellDotConfig.scala

@@ -23,6 +23,8 @@ import com.fasterxml.jackson.annotation.{JsonProperty, JsonPropertyDescription}
 import com.kjetland.jackson.jsonSchema.annotations.{JsonSchemaInject, JsonSchemaTitle}
 import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 
+import javax.validation.constraints.NotNull
+
 @JsonSchemaInject(json = """
 {
   "attributeTypeRules": {
@@ -38,6 +40,7 @@ class DumbbellDotConfig {
   @JsonSchemaTitle("Dot Column Value")
   @JsonPropertyDescription("value for dot axis")
   @AutofillAttributeName
+  @NotNull(message = "Dot Column Value cannot be empty")
   var dotValue: String = ""
 
 }

common/workflow-operator/src/main/scala/org/apache/amber/operator/visualization/dumbbellPlot/DumbbellPlotOpDesc.scala

@@ -29,6 +29,7 @@ import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 import org.apache.amber.operator.metadata.{OperatorGroupConstants, OperatorInfo}
 
 import java.util
+import javax.validation.constraints.{NotBlank, NotNull}
 import scala.jdk.CollectionConverters.CollectionHasAsScala
 //type constraint: measurementColumnName can only be a numeric column
 @JsonSchemaInject(json = """
@@ -46,28 +47,33 @@ class DumbbellPlotOpDesc extends PythonOperatorDescriptor {
   @JsonSchemaTitle("Category Column Name")
   @JsonPropertyDescription("the name of the category column")
   @AutofillAttributeName
+  @NotNull(message = "Category Column Name cannot be empty")
   var categoryColumnName: String = ""
 
   @JsonProperty(value = "dumbbellStartValue", required = true)
   @JsonSchemaTitle("Dumbbell Start Value")
   @JsonPropertyDescription("the start point value of each dumbbell")
+  @NotBlank(message = "Dumbbell Start Value cannot be empty")
   var dumbbellStartValue: String = ""
 
   @JsonProperty(value = "dumbbellEndValue", required = true)
   @JsonSchemaTitle("Dumbbell End Value")
   @JsonPropertyDescription("the end value of each dumbbell")
+  @NotBlank(message = "Dumbbell End Value cannot be empty")
   var dumbbellEndValue: String = ""
 
   @JsonProperty(value = "measurementColumnName", required = true)
   @JsonSchemaTitle("Measurement Column Name")
   @JsonPropertyDescription("the name of the measurement column")
   @AutofillAttributeName
+  @NotNull(message = "Measurement Column Name cannot be empty")
   var measurementColumnName: String = ""
 
   @JsonProperty(value = "comparedColumnName", required = true)
   @JsonSchemaTitle("Compared Column Name")
   @JsonPropertyDescription("the column name that is being compared")
   @AutofillAttributeName
+  @NotNull(message = "Compared Column Name cannot be empty")
   var comparedColumnName: String = ""
 
   @JsonProperty(value = "dots", required = false)

common/workflow-operator/src/main/scala/org/apache/amber/operator/visualization/filledAreaPlot/FilledAreaPlotOpDesc.scala

@@ -28,18 +28,22 @@ import org.apache.amber.operator.PythonOperatorDescriptor
 import org.apache.amber.operator.metadata.annotations.AutofillAttributeName
 import org.apache.amber.operator.metadata.{OperatorGroupConstants, OperatorInfo}
 
+import javax.validation.constraints.NotNull
+
 class FilledAreaPlotOpDesc extends PythonOperatorDescriptor {
 
   @JsonProperty(required = true)
   @JsonSchemaTitle("X-axis Attribute")
   @JsonPropertyDescription("The attribute for your x-axis")
   @AutofillAttributeName
+  @NotNull(message = "X-axis Attribute cannot be empty")
   var x: String = ""
 
   @JsonProperty(required = true)
   @JsonSchemaTitle("Y-axis Attribute")
   @JsonPropertyDescription("The attribute for your y-axis")
   @AutofillAttributeName
+  @NotNull(message = "Y-axis Attribute cannot be empty")
   var y: String = ""
 
   @JsonProperty(required = false)