kz930 opened a new pull request, #7260:
URL: https://github.com/apache/texera/pull/7260

   ### What changes were proposed in this pull request?
   
   `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.`
   
   ### 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.
   
   ### How was this patch tested?
   
   `WorkflowOperator/compile`, `scalafmtCheckAll`, and both operators' 
descriptor specs (12 tests). Each generated module was also executed against an 
empty frame, before and after, with only the `pytexera` import seam stubbed — 
the before run reproduces the `AttributeError`, the after run returns the 
message.
   
   Closes #7244
   
   Generated-by: Claude Code (Claude Opus 5)
   


-- 
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]

Reply via email to