The GitHub Actions job "Required Checks" on 
texera.git/gh-readonly-queue/main/pr-7567-86dd852085db10e8fe7c2db59a9232ef8347eb67
 has failed.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
76ad40d5a0013a8f9cf610d0b33d0e9254042342 / Kary Zheng 
<[email protected]>
feat(visualization): drop rows with missing values before laying out a Dumbbell 
Plot (#7567)

### What changes were proposed in this PR?

Dumbbell Plot took the distinct values of the compared column and sorted
them. An empty cell reaches the operator as `None`, so `unique()`
returned a list with a `None` in it and `sorted` compared `None` against
a string, ending the run with a TypeError.

An empty value is ordinary input here. A blank CSV cell arrives as null,
since univocity returns null for an empty field and
`AttributeTypeUtils.parseField` passes it through by design.

It now drops rows missing any of the three columns it reads, which is
what the other visualization operators do: twenty-four of them open
their generated Python with `dropna(subset=[...]) #remove missing
values`, and Dumbbell Plot was the one that did not. A table left empty
by the drop renders the operator's own error rather than an exception,
matching how it already reports an empty input.

### Any related issues, documentation, discussions?

Closes #7562

### How was this PR tested?

`DumbbellPlotOpDescSpec` gains a case asserting the generated Python
drops on all three configured columns before it sorts. It fails on the
previous behavior, 7 passed / 1 failed before the change and 8 / 0
after.

### Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 5)

Co-authored-by: Claude Opus 5 (1M context) <[email protected]>

Report URL: https://github.com/apache/texera/actions/runs/31673576730

With regards,
GitHub Actions via GitBox

Reply via email to