gnodet-bot commented on code in PR #29569:
URL: https://github.com/apache/camel/pull/29569#discussion_r4526028727
##########
components/camel-mail/src/main/java/org/apache/camel/component/mail/NowSearchTerm.java:
##########
@@ -30,7 +30,6 @@
public class NowSearchTerm extends ComparisonTerm {
private static final long serialVersionUID = 1L;
Review Comment:
⚠️ **Serialization break without `serialVersionUID` bump** — removing the
`private final int comparison` field changes the serialized form of this class.
Any instance serialized before this fix will throw `InvalidClassException` on
deserialization after it. While `NowSearchTerm` is an ephemeral search term
that is unlikely to be persisted across JVM boundaries in practice, the
`serialVersionUID` should be bumped to `2L` to make the incompatibility
explicit and prevent silent corruption in edge cases.
```suggestion
private static final long serialVersionUID = 2L;
```
--
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]
