[
https://issues.apache.org/jira/browse/ANY23-311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16086015#comment-16086015
]
ASF GitHub Bot commented on ANY23-311:
--------------------------------------
GitHub user jgrzebyta opened a pull request:
https://github.com/apache/any23/pull/43
ANY23-311 Simplify rdf graph structure
Fix ANY23-311
Yaml parser produced overly complicated rdf graph so I have done in the PR:
- if yaml document contains a map than the document become the map
- if String is a valid IRI than convert to IRI
- add additional unit test for parsing tree
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jgrzebyta/any23 ANY23-311
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/any23/pull/43.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #43
----
commit daf9b38f249822ab424517cace77530daf08b093
Author: Jacek Grzebyta <[email protected]>
Date: 2017-07-13T17:09:39Z
Simplify rdf graph structure
- if yaml document contains a map than the document become the map
- if String is a valid IRI than convert to IRI
- add additional unit test for parsing tree
Signed-off-by:Jacek Grzebyta <[email protected]>
----
> YAML pareser returns overly complicated RDF tree
> ------------------------------------------------
>
> Key: ANY23-311
> URL: https://issues.apache.org/jira/browse/ANY23-311
> Project: Apache Any23
> Issue Type: Improvement
> Components: rover
> Affects Versions: 2.1
> Reporter: Jacek
> Priority: Minor
>
> If yaml:document contains a map inside than there is necessary node between
> the document and the map.
> {code}
> key1: val1
> key2: val2
> key3: val3
> {code}
> is parsed to: {{<urn:test#document_0> yaml:contains <urn:test#node_1> .}}
> and {{node_1}} contains the map. It might be simplified by removing that
> additional node.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
