The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-5954-810d6da834afb00ff230efd06631a98a32772d1c has succeeded. Run started by GitHub user Yicong-Huang (triggered by Yicong-Huang).
Head commit for run: d522e4e0979e01b949c5bce3deca4bb7a26efdb2 / Xinyuan Lin <[email protected]> test(workflow-operator): add unit test coverage for Sklearn training NB & tree descriptors (#5954) ### What changes were proposed in this PR? Pin behavior of eight previously-untested Sklearn **training** operator descriptors in `common/workflow-operator`. No production-code changes. | Spec | Source class | | --- | --- | | `SklearnTrainingBernoulliNaiveBayesOpDescSpec` | `SklearnTrainingBernoulliNaiveBayesOpDesc` | | `SklearnTrainingComplementNaiveBayesOpDescSpec` | `SklearnTrainingComplementNaiveBayesOpDesc` | | `SklearnTrainingGaussianNaiveBayesOpDescSpec` | `SklearnTrainingGaussianNaiveBayesOpDesc` | | `SklearnTrainingMultinomialNaiveBayesOpDescSpec` | `SklearnTrainingMultinomialNaiveBayesOpDesc` | | `SklearnTrainingDecisionTreeOpDescSpec` | `SklearnTrainingDecisionTreeOpDesc` | | `SklearnTrainingExtraTreeOpDescSpec` | `SklearnTrainingExtraTreeOpDesc` | | `SklearnTrainingExtraTreesOpDescSpec` | `SklearnTrainingExtraTreesOpDesc` | | `SklearnTrainingRandomForestOpDescSpec` | `SklearnTrainingRandomForestOpDesc` | **Behavior pinned** (shared `SklearnTrainingOpDesc` contract) | Surface | Contract | | --- | --- | | `operatorInfo` | exact model name (`Training: <model>`) + `Sklearn <name> Operator` description; **Sklearn Training** group; single `training` input port; one blocking output | | field defaults | `countVectorizer`/`tfidfTransformer` `false`; `target`/`text` `null` | | `getOutputSchemas` | `model_name` (STRING) + `model` (BINARY) keyed by the declared output port | | `generatePythonCode` | imports the matching sklearn estimator and builds the `make_pipeline(...).fit(X, Y)` training model | | 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 (the training-side counterpart to the Sklearn classifier coverage in #5925/#5939/#5940/#5941/#5945/#5946/#5951). ### How was this PR tested? - `sbt "WorkflowOperator/testOnly org.apache.texera.amber.operator.sklearn.training.*"` — 40 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/28332199883 With regards, GitHub Actions via GitBox
