The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-7260-6daa84641094185690ec1d721f91de29635ebd93 has failed. Run started by GitHub user xuang7 (triggered by xuang7).
Head commit for run: a96bf3e70c0e9d5bf72402637633b36e99049a25 / Kary Zheng <[email protected]> fix(visualization): define the render_error the table charts already call (#7260) ### What changes were proposed in this PR? `TablesPlotOpDesc` and `FigureFactoryTableOpDesc` both generate a `TableChartOperator` that calls `self.render_error(...)` on two branches, but neither generated class defines that method. Each now defines it, in the same shape the other visualization operators use. ### Why are the changes needed? Both branches are reachable — an empty input table, and a value column left with only non-positive or null values. Executing each operator's generated module against an empty frame raises `AttributeError: 'TableChartOperator' object has no attribute 'render_error'` instead of showing the message the code was written to show. With the definition added, the same run yields `Tables Plot is not available. Reason is: input table is empty.` ### Any related issues, documentation, discussions? Closes #7244 ### How was this PR tested? `WorkflowOperator/scalafmtCheckAll` and both operators' descriptor specs (14 tests). Each spec now asserts its generated class defines `render_error` and formats the operator's name. Separately, each generated module was run locally against an empty frame before and after the fix, with the `pytexera` and `plotly` imports stubbed — the before run reproduces the `AttributeError`, the after run returns `Tables Plot is not available. Reason is: input table is empty.` ### Does this PR introduce any user-facing change? Yes. Those two cases now render the intended message instead of failing the operator. Nothing changes on the path that produces a chart. ### 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: Xuan Gu <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/31210217656 With regards, GitHub Actions via GitBox
