search

walidazizi / rdflib

Automatically exported from code.google.com/p/rdflib
Other
0 stars 0 forks source link

Reworking of Query result handling #158

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Purpose of code changes on this branch:

(note there is no branch, only attached patch)

This introduces a richer Result object for queries, it is meant to be backwards 
compatible to the SPARQLResults we already get from the sparql impl in 
rdfextras.

It also introduces plugin points for ResultParsers and ResultSerializers 
A matching patch for rdfextras with parser/serializer support for xml and json 
sparql results is coming shortly.

When reviewing my code changes, please focus on:
Does this break anything? :)

After the review, I'll merge this branch into:
trunk
/trunk

Original issue reported on code.google.com by gromgull on 18 Feb 2011 at 9:49

GoogleCodeExporter commented 8 years ago 
patch

Original comment by gromgull on 18 Feb 2011 at 9:49

GoogleCodeExporter commented 8 years ago 
I forgot to mention - this obviously restricts us to the four types of queries 
defined in the new class, select, ask, describe, construct. I cannot think of 
any, but perhaps someone else has a use-case where this is problematic?

Original comment by gromgull on 18 Feb 2011 at 9:51

GoogleCodeExporter commented 8 years ago 
Clearly it was too late at night for me to remember most of the things I wanted 
to say about this:

This would also also deprecate the QueryResult plugin stuff we have now. I have 
problem finding a scenario where you would want to provide a query processor, 
but NOT a result plugin? Or any where you would want to mix and match? For the 
sparql implementation in rdfextras, the two are very tightly coupled.

A future patch, (not yet written), will make the rdfextras sparql processor 
work with this.

Original comment by gromgull on 18 Feb 2011 at 10:10

GoogleCodeExporter commented 8 years ago 
new more correct patch :) 

See also http://code.google.com/p/rdfextras/source/list?r=query-result-cleanup

Original comment by gromgull on 21 Feb 2011 at 1:40

GoogleCodeExporter commented 8 years ago 
Moved the patch to a hg branch, query-result-cleanup: 

http://code.google.com/p/rdflib/source/browse/?r=query-result-cleanup

Original comment by gromgull on 30 Mar 2011 at 9:13

GoogleCodeExporter commented 8 years ago 
Ok - I'll give you until next week, when I am back from holiday I will merge if 
noone says anything :)

Original comment by gromgull on 3 Aug 2011 at 8:20

GoogleCodeExporter commented 8 years ago 
This issue was closed by revision d45003524720.

Original comment by gromgull on 18 Aug 2011 at 5:34