[
https://issues.apache.org/jira/browse/OPENNLP-1151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16216021#comment-16216021
]
ASF GitHub Bot commented on OPENNLP-1151:
-----------------------------------------
kottmann commented on a change in pull request #282: OPENNLP-1151: All Sample
objects should implement Serializable for easy integration into other tools
URL: https://github.com/apache/opennlp/pull/282#discussion_r146418457
##########
File path:
opennlp-tools/src/test/java/opennlp/tools/namefind/NameSampleTest.java
##########
@@ -61,6 +71,36 @@ private static NameSample createSimpleNameSample(boolean
useTypes) {
return nameSample;
}
+ @Test
+ public void testNameSampleSerDe() throws IOException {
+ NameSample nameSample = createGoldSample();
+ String fileName = "file.ser";
Review comment:
Maybe better to write this into a ByteArrayOutputStream ?!
----------------------------------------------------------------
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:
[email protected]
> All Sample objects should implement Serializable for easy interation into
> other tools
> -------------------------------------------------------------------------------------
>
> Key: OPENNLP-1151
> URL: https://issues.apache.org/jira/browse/OPENNLP-1151
> Project: OpenNLP
> Issue Type: Bug
> Reporter: Joern Kottmann
> Assignee: Suneel Marthi
> Priority: Minor
> Fix For: 1.8.3
>
>
> State of the Art frameworks like Apache Flink require that objects are
> serializable to use them in the pipeline. To use it to prepare training date
> for OpenNLP the Sample objects should all implement Serializable.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
