Skip to content

Writer updates#263

Merged
asfgit merged 5 commits intoapache:masterfrom
afs:writer-updates
Jun 20, 2017
Merged

Writer updates#263
asfgit merged 5 commits intoapache:masterfrom
afs:writer-updates

Conversation

@afs
Copy link
Copy Markdown
Member

@afs afs commented Jun 17, 2017

JENA-369: Some lists in WHERE blocks are printed in ()-form - in particular, property functions are likely to be done this way.

JENA-1361: Fix bug in update output.

The existing testing already covers the changes in TestSerialziation.

@afs
Copy link
Copy Markdown
Member Author

afs commented Jun 17, 2017

Printing basic graph patterns prettily for queries is a different problem to printing Turtle if it is to retain the triple ordering of the BGP.

This PR conservatively spots the layout the parser generates and puts back pretty lists for at least flat lists.

It does not apply to property paths.

@afs afs force-pushed the writer-updates branch 3 times, most recently from 320cfc4 to f249233 Compare June 18, 2017 08:26
This includes property functions.
Conversion is cautious - it looks for the triples as the parser
might output them.

Nested bnodes structures in lists are not converted back.
@afs afs force-pushed the writer-updates branch from f249233 to f19d8b2 Compare June 18, 2017 08:27
@asfgit asfgit merged commit f19d8b2 into apache:master Jun 20, 2017
asfgit pushed a commit that referenced this pull request Jun 20, 2017
@afs afs deleted the writer-updates branch June 20, 2017 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants