ASF GitHub Bot commented on COMMONSRDF-49:

Github user stain commented on the issue:

    You may have better ideas on how to do something like 
 so it is serializable.  For instance the 
 beans might or might not be serializable depending on the implementation.
    (Surely one that is just connected to an open InputStream is not 
serializable, but one that has just got an IRI should be serializable. I made 
the implementations package private: 
 which meant I had to make a new 
 to avoid Simple dependency)

> Make AbstractRDFParser serializable
> -----------------------------------
>                 Key: COMMONSRDF-49
>                 URL: https://issues.apache.org/jira/browse/COMMONSRDF-49
>             Project: Apache Commons RDF
>          Issue Type: New Feature
>          Components: simple
>    Affects Versions: 0.3.0
>            Reporter: Stian Soiland-Reyes
>            Assignee: Stian Soiland-Reyes
>            Priority: Major
>              Labels: parser
>             Fix For: 0.6.0
> Raised by [~p_ansell] in [pull request 
> 25|(https://github.com/apache/incubator-commonsrdf/pull/25#discussion_r85436754]
> {quote}
> The use of optional here as a field type makes it impossible to serialise. 
> Need to have the raw values stored in fields if you want to support 
> serialisation in the future, which should otherwise be possible.
> {quote}
> The suggestion is to avoid {{Optional}} in the private fields of 
> {{AbstractRDFParser}} so it can be serialized - it can still be {{Optional}} 
> in the accessor methods.

This message was sent by Atlassian JIRA

Reply via email to