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/07/23 01:49:23 UTC
[GH] (texera/gh-readonly-queue/main/pr-6808-0fd0d887fbaca7c12ba7f9585ba8c96be9a95231): Workflow run "Required Checks" failed!
The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-6808-0fd0d887fbaca7c12ba7f9585ba8c96be9a95231 has failed.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run:
c436ab9d6592bbc243697b6448caf08363b87a57 / Kary Zheng <15...@users.noreply.github.com>
fix(LogicalOp): register SklearnLogisticRegression(CV) subtypes exactly once (#6808)
### What changes were proposed in this PR?
`LogicalOp`'s `@JsonSubTypes` registered
`SklearnLogisticRegressionOpDesc` and
`SklearnLogisticRegressionCVOpDesc` **twice each** (two separate active
`new Type(...)` pairs). Any consumer that enumerates
`@JsonSubTypes.value()` — operator discovery, metadata/schema
generation, the operator palette — therefore saw each of these two
operators twice.
This PR removes the duplicate pair, keeping exactly one registration of
each:
```diff
- new Type(value = classOf[SklearnLogisticRegressionOpDesc], name = "SklearnLogisticRegression"),
- new Type(
- value = classOf[SklearnLogisticRegressionCVOpDesc],
- name = "SklearnLogisticRegressionCV"
- ),
```
### Any related issues, documentation, discussions?
Closes #6793
### How was this PR tested?
Added a `LogicalOpSpec` regression test that reads the `@JsonSubTypes`
annotation and asserts no subtype **class** or **name** is registered
more than once. Verified the test **fails** on the pre-fix (duplicated)
registry and **passes** after the fix; full `LogicalOpSpec` is green
(5/5).
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Claude Opus 4.8)
---------
Co-authored-by: Claude Opus 4.8 <no...@anthropic.com>
Report URL: https://github.com/apache/texera/actions/runs/29972176847
With regards,
GitHub Actions via GitBox