Yicong-Huang commented on PR #7104:
URL: https://github.com/apache/texera/pull/7104#issuecomment-5126497574
Resolved as a source-only backport. The fix maps cleanly onto v1.2: the
buggy `schemaOptional.getOrElse(throw new IllegalStateException("Schema is
missing"))` (which discarded the underlying `Left(cause)`) is replaced with the
`match` that surfaces the real cause (#3546). I deliberately dropped the
incoming `reuseStorage`/`createOrReuseDocument` lines that came adjacent in the
hunk — those belong to the main-only loop-storage feature (#5707) and are not
part of this fix, so v1.2 keeps `DocumentFactory.createDocument`. The spec is
left at its v1.2 version (the fix's test additions diverge heavily from v1.2's
termination-retry spec); this follows the established source-fix-only
convention for release backports. Marking ready.
--
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]
