carloea2 opened a new pull request, #8168: URL: https://github.com/apache/texera/pull/8168
### What changes were proposed in this PR? Validate the required Pie Chart Name Column in both generated table-processing stages. An empty name is now rejected before the generated Python reaches pandas. The existing valid chart test remains, and the previous empty-name characterization is now a regression test for both stages. ### Any related issues, documentation, discussions? Closes #8167 ### How was this PR tested? ```shell sbt -no-colors "WorkflowOperator/testOnly org.apache.texera.amber.operator.visualization.pieChart.PieChartOpDescSpec" sbt -no-colors scalafmtCheckAll "scalafixAll --check" sbt -no-colors "WorkflowOperator/console" ``` The focused suite passed 10 tests. For the live check, I first executed the generated pandas `dropna` shape with Value Column set and Name Column empty. It failed with a KeyError for the empty column name. I then invoked the fixed `PieChartOpDesc.manipulateTable` in Texera's Scala console with the same configuration. It rejected the configuration with `Name Column cannot be empty` before Python generation. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Codex GPT-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]
