lukaszlenart commented on a change in pull request #406:
URL: https://github.com/apache/struts/pull/406#discussion_r415537558
##########
File path: plugins/json/src/main/java/org/apache/struts2/json/JSONResult.java
##########
@@ -68,8 +69,6 @@
*/
public class JSONResult implements Result {
- private static final long serialVersionUID = 8624350183189931165L;
Review comment:
Yeah, the idea was as this is a larger change to the class the ID should
be changed, then I thought _why do we need the ID at all here, nobody should
ever serialize a instance of `Result`_. So probably the main case is to stop
extending `Serializable` by `Result`. I will put back a new ID.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]