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

   ### What changes were proposed in this PR?
   
   Collect internal stage errors before hiding internal schemas. Preserve the 
originating validation detail instead of reporting only a later missing-schema 
error. Valid output schemas and invalid-workflow rejection stay unchanged.
   
   Classified as feat because this improves the user-facing diagnostic rather 
than restoring established functionality.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8793
   
   ### How was this PR tested?
   
   Local candidate checks: 26 focused and 66 compiler-module cases passed. 
Three negative cases failed before implementation. Module scalafix and 
repository Scala formatting passed.
   
   ```text
   sbt "WorkflowCompiler/testOnly 
org.apache.texera.common.compiler.WorkflowCompilerSpec"
   sbt "WorkflowCompiler/test"
   sbt "WorkflowCompiler/scalafix --check" "WorkflowCompiler/Test/scalafix 
--check" scalafmtCheckAll
   ```
   
   On localhost, compile the same legacy SUM graph with a missing column: the 
response retains the column error after this change. Its valid counterpart 
keeps the same schema. Chrome at 1707 by 979 also verified the combined example 
below.
   
   These local checks preceded today's rebase onto upstream main 98d3d4b1ec. 
Candidate changes are unchanged. Remote CI and the full operating-system matrix 
are not claimed.
   
   Combined example uses the separate conditional-Aggregate candidate. That 
feature is not part of this compiler PR; independent reproduction used legacy 
SUM.
   
   | Generic diagnostic | Originating diagnostic |
   | --- | --- |
   | ![Generic 
diagnostic](https://github.com/user-attachments/assets/01f6d4fc-1ec7-47b3-a817-3a38ad966321)
 | ![Originating 
diagnostic](https://github.com/user-attachments/assets/04fb1af9-9144-4afb-9205-079d1c9b7a37)
 |
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: OpenAI Codex (exact authoring version not recorded).
   


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