carloea2 commented on PR #8637: URL: https://github.com/apache/texera/pull/8637#issuecomment-5805292442
Thanks for the update, @pedroreyesalavrado. I inspected `0eb0a3f60`: the live-value guard and the new test address the stale model-setting example from my review, and the unchanged-field test now checks the rerun. I have not independently run the tests. Could you add side-by-side before/after screenshots and a short GIF or recording showing the error, Analyze, the suggested diff, Apply and re-run, and the resulting execution? Please also show Discard and an unsupported-error state. Include browser, viewport, commit, and reproduction steps using the supplied demo workflow. Two tracking items: please restore the PR template's `###` headings, and link a feature-specific issue for the AI fixer (or open one if none exists). #8520 is the general hackathon discussion: https://github.com/apache/texera/discussions/8520. Before merging, please also address the documented warehouse-selection case so the UI only reports "re-running" when execution actually starts. -- 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]
