rdflib 3.2.2 sparql query broken

[nschwertner]

Running this example:

https://github.com/RDFLib/rdflib/blob/master/examples/sparql_query_example.py

under Python 2.7 with rdfextras 0.1 and 0.2 produces:

TypeError: query() takes exactly 4 arguments (5 given)

[nschwertner]

It looks like the addition of this method is causing the problem:

rdflib/rdflib/store.py

Line 261 in e98a91b

def query(self, query, initNs, initBindings, **kwargs):

The problem was first introduced with this commit:
ca4085f

[gromgull]

You are right - it IS broken, but not that in that commit - it was broken over 2 years ago:

r451780760e801d51288d62e131ce70137bd1d086

I'm surprised noone tried to query since then...

I will fix and push a 3.2.3 soon

[gromgull]

Hmm - it was broken in 3.2.1 as well - and noone noticed :)

[gromgull]

Hmm - tricker than I thought - in sparql_store I actually use the signature:

query(self, graph, query, initNs, initBindings, **kwargs) 

since you need to know whether the requesting graph is a ConjunctiveGraph (in which case you should query the union of all graphs, depending on the DAWG_COMPLIANCE_FLAG)
This also matches what a real QueryProcessor sees - it gets constructed with the graph instance:

https://github.com/RDFLib/rdflib/blob/master/rdflib/graph.py#L936

BUT mixing the graph into the store interface and requiring the store implementor to know about conjunctive graphs seems too complicated - so I've added a new queryGraph parameter, see docs in store.query.

[gromgull]

A new 3.2.3 has been release with this fix.

[nschwertner]

Thank you, Gunnar. I tested 3.2.3 with our code and now everything is back to normal. Confirming successful resolution of the issue.