You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@commons.apache.org by stain <gi...@git.apache.org> on 2018/02/13 01:03:22 UTC

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

Github user stain commented on the issue:

    https://github.com/apache/commons-rdf/pull/43
  
    Picking up this - @ajs6f do you still think we should proceed along those lines? I am also reluctant to making the abstract factory (builder) serializable, but I can see the reasoning, particularly if you want to use this in Hadoop or something where you have a pre-made parser builder and then tell a different node to run it.
    
    One thing I feel I need to check more is that there is no reading of the now-usually-null fields beyond the getters - I might rename them to make that clear.
    
    I have put this PR into upstream branch COMMONSRDF-49 we can contribute to.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org