[ 
https://issues.apache.org/jira/browse/OPENNLP-1146?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16209369#comment-16209369
 ] 

ASF GitHub Bot commented on OPENNLP-1146:
-----------------------------------------

kottmann commented on issue #277: OPENNLP-1146: remove unnecessary 
serialVersionUID
URL: https://github.com/apache/opennlp/pull/277#issuecomment-337601383
 
 
   The default is a reasonable value. The idea is that you can change this in 
case you have a use case of serializing and deserializing objects from 
different versions of OpenNLP. This is something our users really should not do.
   
   And if they do it anyway, and we want to support that we can, by setting the 
version to indicate backward compatibility has been broken.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> remove unnecessary serialVersionUID
> -----------------------------------
>
>                 Key: OPENNLP-1146
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1146
>             Project: OpenNLP
>          Issue Type: Improvement
>          Components: Build, Packaging and Test
>    Affects Versions: 1.8.2
>            Reporter: Koji Sekiguchi
>            Assignee: Koji Sekiguchi
>            Priority: Trivial
>             Fix For: 1.8.3
>
>
> We saw several classes that have unnecessary serialVersionUID constant 
> declaration. Most of them are Stemmer classes that are created by the 
> Snowball to Java compiler. I think we can just remove serialVersionUID from 
> Stemmer classes. Other than Stemmer classes, Exception classes which extend 
> RuntimeException or IOException have serialVersionUID. I'll remove 
> serialVersionUID from these Exception classes as well but add 
> @SuppressWarnings("serial") just in case.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to