The GitHub Actions job "Required Checks" on texera.git/release/v1.2 has 
succeeded.
Run started by GitHub user github-merge-queue[bot] (triggered by 
github-merge-queue[bot]).

Head commit for run:
9ef3744deb816c99d4f92e1dc9ed4c915c542040 / Kary Zheng 
<[email protected]>
fix(visualization, v1.2): take the union of Network Graph's two node columns 
(#7518)

### What changes were proposed in this PR?

Backport of #7327 to `release/v1.2`, superseding #7401. That PR's branch
lives on `apache/texera` rather than a fork, so the conflict resolution
could not be pushed to it.

The operator change is #7327's, unchanged. Network Graph built its node
set with `set(sources + destinations)`; on two pandas Series `+` is
element-wise, so the set held each source glued to its destination
rather than the union of the two columns. This takes the union in
first-appearance order.

The cherry-pick's one conflict was `NetworkGraphOpDescSpec.scala`, which
does not exist on `release/v1.2` — it was created on main by #5640,
which was never backported. Resolving that modify/delete by taking the
whole file, which is what #7401 carries, brings across three cases
asserting the assertion text names the empty field. This branch still
has a bare `assert(source.nonEmpty)`, so those three fail here: running
#7401's branch as it stands gives three passed and three failed. This
backport instead carries only the case #7327 added, so what lands is the
one-line operator fix and the test that pins it.

### Any related issues, documentation, discussions?

Backport of #7327. Originally linked #7325. Supersedes #7401, which can
be closed.

### How was this PR tested?

`NetworkGraphOpDescSpec` on this branch: one case, passing. Reverting
the operator line leaves it failing. `WorkflowOperator/scalafmtCheckAll`
and `WorkflowOperator/scalafixAll --check` are both clean.

The operator change is byte-identical to #7327, where it was verified by
dumping and executing the generated module over rows carrying known
edges.

### 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]>
Co-authored-by: Xinyuan Lin <[email protected]>

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

With regards,
GitHub Actions via GitBox

Reply via email to