DanielLeens commented on PR #12208:
URL: https://github.com/apache/seatunnel/pull/12208#issuecomment-6397571594

   Thanks for the contribution. My previous review described queued checks. The 
exact-head [fork 
Build](https://github.com/88fantasy/seatunnel/actions/runs/39750960377) is now 
**failed**, so that pending-status guidance is obsolete.
   
   I inspected the [Java 11 Linux unit-test 
log](https://github.com/88fantasy/seatunnel/actions/runs/39750960377/job/119732244124):
 `LLMBooleanOutputTest.readsLegacyFactoryWrapperWithValidationDisabled` fails 
with `InvalidClassException` for `AbstractMultiCatalogTransform`, comparing 
archived UID `7090258061839605137` with local UID `-3241928148724104003`. 
Latest dev already fixes this exact archived-wrapper contract in 
[#12938](https://github.com/apache/seatunnel/pull/12938), commit 
`cc54bb433357b4cb45db9cd5d0743589c3111b79`, which explicitly restores that UID. 
I checked that this head does not contain the fix.
   
   There is a separate [transform-v2 part-2 
failure](https://github.com/88fantasy/seatunnel/actions/runs/39750960377/job/119732244753):
 `TestPythonTransformIT.testMultiTablePythonTransform` fails during container 
setup because Ubuntu's package index download reports `Hash Sum mismatch`. That 
is not a lineage assertion. I did not find a matching committed fix in the 
current Python fixture history, and the UID fix is not claimed to solve a 
package-mirror error.
   
   The comparison is `diverged`, ahead 15 / behind 27 against dev, with no 
conflict. Please sync latest dev to incorporate #12938, then rerun the affected 
failed lanes. If the package-index failure repeats, retain its new job link for 
a separate fixture/mirror investigation; no unrelated package or transform 
patch belongs in this lineage PR merely to chase it.
   
   My earlier full source conclusion remains clear of blockers. This is a 
CI-fact correction, not a new full source review or an effective approval. 
Required Build and a write-capable maintainer's final approval remain 
outstanding. Thank you for the contribution.
   


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