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

Head commit for run:
6de37efa301b4635e3f5189d8872b7b450dca982 / 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/31672951595

With regards,
GitHub Actions via GitBox

Reply via email to