aglinxinyuan opened a new pull request, #5897:
URL: https://github.com/apache/texera/pull/5897
### What changes were proposed in this PR?
Pin behavior of two previously-untested visualization plot descriptors in
`common/workflow-operator`. No production-code changes.
| Spec | Source class | Tests |
| --- | --- | --- |
| `QuiverPlotOpDescSpec` | `QuiverPlotOpDesc` | 5 |
| `RadarPlotOpDescSpec` | `RadarPlotOpDesc` | 6 |
**Behavior pinned**
| Surface | Contract |
| --- | --- |
| `operatorInfo` | exact name + description; Scientific group; 1-in/1-out |
| field defaults | Quiver `x/y/u/v` empty; Radar booleans default `true`,
optional columns empty, `selectedAttributes`/`linePattern` null |
| `getOutputSchemas` | single `html-content` STRING column keyed by the
declared output port |
| `generatePythonCode` | emits the Plotly figure (`ff.create_quiver(` /
`go.Scatterpolar`) and carries the configured columns; Radar requires
`linePattern` (dereferenced without a null guard) |
| Round-trip | config fields preserved through the polymorphic `LogicalOp`
base |
Note: column fields are `EncodableString`, so in the emitted (encoded) code
they appear as `self.decode_python_template('<base64>')` — assertions use a
`carries` helper that accepts the raw name or its base64 form.
### Any related issues, documentation, discussions?
Part of the ongoing `workflow-operator` unit-test coverage effort (follow-up
to #5843, #5844).
### How was this PR tested?
- `sbt "WorkflowOperator/testOnly *QuiverPlotOpDescSpec
*RadarPlotOpDescSpec"` — 11 tests, all green
- `sbt "WorkflowOperator/Test/scalafmtCheck"` and `sbt
"WorkflowOperator/scalafixAll --check"` — clean
- CI to confirm
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Opus 4.8 [1M context])
--
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]
