Ma77Ball opened a new pull request, #5616:
URL: https://github.com/apache/texera/pull/5616
### What changes were proposed in this PR?
Backport of #5416 ("fix(workflow-operator): prevent NPE when Sort operator
is unconfigured") to `release/v1.2`.
Applies as a clean `git cherry-pick -x` of the merged squash commit
`5f542752`, no conflicts and no prerequisite chain. 2 files changed, 72
insertions, 1 deletion, identical to the original.
For the full change description see #5416. In brief: `SortOpDesc` threw a
`NullPointerException` when the operator's sort attributes were left
unconfigured; this guards that path so an unconfigured Sort operator fails
gracefully instead of NPE-ing.
### Any related issues, documentation, discussions?
Backports #5416 (which closes #3191 on `main`). Requested by @xuang7 for the
v1.2 release; the PR was merged before the `release/v1.2` label existed, so
auto-backport did not trigger.
### How was this PR tested?
This is a backport with no changes beyond the single cherry-picked commit,
so it relies on the existing tests carried over from #5416 (added
`SortOpDescSpec` cases). Run them with `sbt "workflowOperator/testOnly
*SortOpDescSpec"` from the repo root.
Backport fidelity was verified locally: after the cherry-pick, every file
touched by #5416 is byte-identical to its state on `main` at the merged commit
(`5f542752`), and `release/v1.2` had no independent changes to any of those
files.
Full compile and unit-test runs are left to CI.
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Claude Opus 4.8)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
