Ma77Ball opened a new pull request, #4919:
URL: https://github.com/apache/texera/pull/4919
### What changes were proposed in this PR?
LineChartOpDesc declared var lines: util.List[LineConfig] = _, which
defaults to null in Scala/Java. Calling generatePythonCode() on a freshly
constructed instance immediately crashed with a NullPointerException at
createPlotlyFigure because .asScala.map(...) was called on a null
reference.
Two changes:
- Default lines to new util.ArrayList[LineConfig]() instead of null, so it
is always safe to iterate.
- Add assert(lines.asScala.nonEmpty, "At least one line must be
configured") at the top of createPlotlyFigure(), matching the explicit
assertion pattern used by sibling visualizers (HeatMapOpDesc, BarChartOpDesc).
### Any related issues, documentation, or discussions?
#4811
### How was this PR tested?
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude opus 4.6 in compliance with ASF
--
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]
