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

Marshall Schor commented on UIMA-4685:
--------------------------------------

Hi, in designing APIs for others to use, I try to imagine if it is more likely 
a "normal" operation (to pass a null sometimes) or that is more likely a 
mistake, which the user would appreciate knowing about.

In this case, I guessed (true, it's a guess... maybe wrong) that it would be 
more useful to users who explicitly wrote this call with a type system 
parameter, to know it was null, than to have the other default functionality.  
I don't feel strongly one way or another so if you feel it's better the other 
way, I'm willing to change it.

> Add Common Load/Save APIs, including type system and index defs, for various 
> CAS formats
> ----------------------------------------------------------------------------------------
>
>                 Key: UIMA-4685
>                 URL: https://issues.apache.org/jira/browse/UIMA-4685
>             Project: UIMA
>          Issue Type: New Feature
>          Components: CasEditor, Core Java Framework
>    Affects Versions: 2.8.1SDK
>            Reporter: Peter Klügl
>            Assignee: Marshall Schor
>             Fix For: 2.9.0SDK
>
>         Attachments: BinaryCasIOUtils.java, caseditor.csv, 
> caseditor_bincas.csv
>
>
> Create a new CasIOUtils class having common load and save APIs, with optional 
> saving of type system and index definitions, in all the various CAS 
> serialization formats (except JSON).  Implement heuristic discovery for 
> deserialization to support automatic selection of the right kind of 
> deserializer.  Update the headers for binary to support indicating they have 
> the type system included (not used yet).  Update the SerialFormat to support 
> this.  Update the CasEditor and other components to use this new API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to