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

   ### What changes were proposed in this PR?
   
   Drain the small UTF-16 fixture in finally so the lazy reader closes before 
file deletion. Keep the decoding assertions and exhausted-iterator check. This 
repairs one test; no production behavior changes.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8795
   
   ### How was this PR tested?
   
   On Windows: baseline 13 passed and one failed; patched 14 passed. A 
temporary wrong-encoding mutation still failed the decoding assertion. The scan 
suites passed 163 tests with one existing S3 fixture cancellation. Formatting 
and lint passed.
   
   ```text
   sbt "WorkflowOperator/testOnly 
org.apache.texera.amber.operator.source.scan.file.FileScanSourceOpDescSpec"
   sbt "WorkflowOperator/testOnly 
org.apache.texera.amber.operator.source.scan.*"
   sbt "WorkflowOperator/Test/scalafix --check" scalafmtCheckAll
   ```
   
   Localhost Chrome at 1707 by 979 also ran the unchanged UTF-16 reader and 
returned the three expected lines.
   
   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.
   
   Test-only change. This screenshot is a compatibility check, not a visual 
before and after.
   
   ![Unchanged reader 
compatibility](https://github.com/user-attachments/assets/01390900-58d3-46ae-abcb-54884afe0269)
   
   ### 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