SEZ9 commented on issue #12929:
URL: https://github.com/apache/seatunnel/issues/12929#issuecomment-6332034246

   Thanks for the detailed analysis — the stack trace, the 
`src/test/resources/llm/legacy-boolean-wrapper.base64` fixture path, and the 
`serialver` comparison make the root cause clear: the implicit 
`serialVersionUID` of `AbstractMultiCatalogTransform` changed when #12804 added 
fields and methods to the serializable superclass without an explicit UID.
   
   As noted above, the compatibility decision (pinning the pre-#12804 UID 
versus regenerating the fixture and narrowing the test to the LLM leaf class) 
is being handled in #12938, so please keep that discussion there rather than 
opening a competing change here. Separately, it may be worth considering 
explicit `serialVersionUID` declarations on other Serializable transform base 
classes as a follow-up, so a future structural change cannot silently break 
archived streams again.
   
   <!-- streview-comment:1949 -->


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