officialasishkumar opened a new pull request, #5177:
URL: https://github.com/apache/texera/pull/5177

   ### What changes were proposed in this PR?
   
   Some operator boolean properties initialized to `false` did not declare a 
JSON-schema default value, so the frontend form could render them as neutral 
checkboxes instead of unchecked checkboxes.
   
   This PR adds explicit `defaultValue = "false"` metadata for:
   
   - `FilledAreaPlotOpDesc.facetColumn`
   - `DumbbellPlotOpDesc.showLegends`
   - `JSONLScanSourceOpDesc.flatten`
   
   It also adds a focused metadata-generation spec that asserts these boolean 
schemas include a `false` default.
   
   ### Any related issues, documentation, discussions?
   
   Closes #4015.
   
   ### How was this PR tested?
   
   Validated locally with a temporary Postgres container loaded with the same 
DDL files used by CI for JOOQ generation.
   
   ```
   JAVA_HOME=/usr/lib/jvm/java-17-openjdk-amd64 
COURSIER_CACHE=/tmp/texera-coursier-cache 
/usr/lib/jvm/java-17-openjdk-amd64/bin/java 
-Dsbt.ivy.home=/tmp/texera-ivy-cache -Dsbt.boot.directory=/tmp/texera-sbt-boot 
-Dsbt.global.base=/tmp/texera-sbt-global -jar /tmp/sbt-launch-1.12.9.jar 
"WorkflowOperator/testOnly 
org.apache.texera.amber.operator.metadata.OperatorBooleanDefaultSpec"
   ```
   
   Result: 2 passed.
   
   ```
   JAVA_HOME=/usr/lib/jvm/java-17-openjdk-amd64 
COURSIER_CACHE=/tmp/texera-coursier-cache 
/usr/lib/jvm/java-17-openjdk-amd64/bin/java 
-Dsbt.ivy.home=/tmp/texera-ivy-cache -Dsbt.boot.directory=/tmp/texera-sbt-boot 
-Dsbt.global.base=/tmp/texera-sbt-global -jar /tmp/sbt-launch-1.12.9.jar 
scalafmtCheckAll
   ```
   
   Result: passed.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: OpenAI Codex (GPT-5)
   


-- 
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]

Reply via email to