You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@texera.apache.org by GitBox <gi...@apache.org> on 2026/06/26 23:25:41 UTC
[GH] (texera/gh-readonly-queue/main/pr-5952-363308bfa7a880f4e33fbfc42b35e51d2e044c6a): Workflow run "Required Checks" is working again!
The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-5952-363308bfa7a880f4e33fbfc42b35e51d2e044c6a has succeeded.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run:
2ec7c076cb41c840bd052d8b86f4ac93c05d974c / Xinyuan Lin <xi...@uci.edu>
test(workflow-operator): add unit test coverage for Sklearn prediction/testing/linear-regression descriptors (#5952)
### What changes were proposed in this PR?
Pin behavior of three previously-untested Sklearn model-lifecycle
descriptors in `common/workflow-operator`. No production-code changes.
Unlike the classifier family, these extend `PythonOperatorDescriptor`
directly, so each has its own contract.
| Spec | Source class | Tests |
| --- | --- | --- |
| `SklearnLinearRegressionOpDescSpec` | `SklearnLinearRegressionOpDesc`
| 5 |
| `SklearnPredictionOpDescSpec` | `SklearnPredictionOpDesc` | 5 |
| `SklearnTestingOpDescSpec` | `SklearnTestingOpDesc` | 6 |
**Behavior pinned**
| Surface | Contract |
| --- | --- |
| `operatorInfo` | exact name/description; Sklearn group; correct
multi-port shapes (LinearRegression training/testing + blocking output;
Prediction model/data; Testing model/data) |
| field defaults | LinearRegression `target` null / `degree` 1;
Prediction `model`/`resultAttribute` null, `groundTruthAttribute` `""`;
Testing `isRegression` false, `model`/`target` null |
| `getOutputSchemas` | LinearRegression → `model_name`/`model`;
Prediction → input (port 1) + result attribute; Testing → input (port 0)
+ classification (`accuracy`/`f1`/`precision`/`recall`) or regression
(`R2`/`RMSE`/`MAE`) metric columns |
| `generatePythonCode` | emits the expected estimator import / pipeline
/ scorer fragments |
| Round-trip | config fields preserved through the polymorphic
`LogicalOp` base, with the correct `operatorType` discriminator |
### Any related issues, documentation, discussions?
Part of the ongoing `workflow-operator` unit-test coverage effort.
### How was this PR tested?
- `sbt "WorkflowOperator/testOnly *SklearnLinearRegressionOpDescSpec
*SklearnPredictionOpDescSpec *SklearnTestingOpDescSpec"` — 16 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])
Report URL: https://github.com/apache/texera/actions/runs/28270423371
With regards,
GitHub Actions via GitBox