Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QueryJoinOptimizer breaks ordering of dependent statements #83

Closed
abrokenjester opened this issue May 11, 2016 · 1 comment
Closed

QueryJoinOptimizer breaks ordering of dependent statements #83

abrokenjester opened this issue May 11, 2016 · 1 comment
Labels
🐞 bug issue is a bug specification issues related to compliance to standards and external specs
Milestone

Comments

@abrokenjester
Copy link
Contributor

(Migrated from https://openrdf.atlassian.net/browse/SES-2275 )

Test case bind03 passes by happy coincidence rather than by design (attached for convenience).
Currently, to evaluate
{code}
PREFIX : http://example.org/

SELECT ?z ?s1
{
?s ?p ?o .
BIND(?o+1 AS ?z)
?s1 ?p1 ?z
}
{code}
depends on the order in which the statements are passed to the evaluation engine. But the QueryJoinOptimizer can rearrange the order. In this case, the order is unchanged, but the following fails:
{code}
PREFIX : http://example.org/

SELECT ?z ?s1
{
?s ?p ?o .
BIND(?o+1 AS ?z)
?s1 :p ?z
}
{code}
whereas this works:
{code}
PREFIX : http://example.org/

SELECT ?z ?s1
{
?s :p ?o .
BIND(?o+1 AS ?z)
?s1 ?p1 ?z
}
{code}
.
Should sparql be working out the dependency ordering or should it respect the ordering given by the user? i.e. should this work or not:
{code}
PREFIX : http://example.org/

SELECT ?z ?s1
{
?s1 ?p1 ?z
BIND(?o+1 AS ?z)
?s :p ?o .
}
{code}

@abrokenjester abrokenjester added the 🐞 bug issue is a bug label May 11, 2016
@catch-point catch-point added the specification issues related to compliance to standards and external specs label Jun 23, 2017
@catch-point catch-point added this to the 2.2.2 milestone Aug 29, 2017
@catch-point
Copy link

Duplicate of #380

@catch-point catch-point marked this as a duplicate of #380 Aug 29, 2017
abrokenjester pushed a commit that referenced this issue Aug 22, 2019
Removed unused imports and imports from same package
abrokenjester pushed a commit that referenced this issue Aug 22, 2019
…rsion

Features/rdf4j#461project parent pom streamlining
abrokenjester pushed a commit that referenced this issue Jul 25, 2020
Fix #1224 replaced old inferencer with new in docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug issue is a bug specification issues related to compliance to standards and external specs
Projects
None yet
Development

No branches or pull requests

2 participants