aglinxinyuan opened a new pull request, #5898:
URL: https://github.com/apache/texera/pull/5898
### What changes were proposed in this PR?
Pin behavior of four previously-untested Hugging Face operator descriptors
in `common/workflow-operator`. No production-code changes.
| Spec | Source class | Tests |
| --- | --- | --- |
| `HuggingFaceSentimentAnalysisOpDescSpec` |
`HuggingFaceSentimentAnalysisOpDesc` | 7 |
| `HuggingFaceSpamSMSDetectionOpDescSpec` |
`HuggingFaceSpamSMSDetectionOpDesc` | 6 |
| `HuggingFaceTextSummarizationOpDescSpec` |
`HuggingFaceTextSummarizationOpDesc` | 7 |
| `HuggingFaceIrisLogisticRegressionOpDescSpec` |
`HuggingFaceIrisLogisticRegressionOpDesc` | 6 |
**Behavior pinned**
| Surface | Contract |
| --- | --- |
| `operatorInfo` | exact name + description; Hugging Face group; 1-in/1-out
(Sentiment also `supportReconfiguration`) |
| field defaults | all column fields default to `null` |
| `getOutputSchemas` | appended result-column types (Sentiment 3×DOUBLE;
Spam BOOLEAN+DOUBLE; Summarization STRING; Iris STRING+DOUBLE) keyed by the
declared output port, plus the null/blank result-name guards |
| `generatePythonCode` | emits the model id + structural fragments and
carries the configured columns |
| `getPhysicalOp` | wires `OpExecWithCode` tagged `"python"` and carries the
operator's port identities |
| 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 *HuggingFaceSentimentAnalysisOpDescSpec
*HuggingFaceSpamSMSDetectionOpDescSpec *HuggingFaceTextSummarizationOpDescSpec
*HuggingFaceIrisLogisticRegressionOpDescSpec"` — 26 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]
