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

Issue with Nested Laterals #2924

Closed
Aklakan opened this issue Jan 7, 2025 · 0 comments · Fixed by #2925
Closed

Issue with Nested Laterals #2924

Aklakan opened this issue Jan 7, 2025 · 0 comments · Fixed by #2925
Labels

Comments

@Aklakan
Copy link
Contributor

Aklakan commented Jan 7, 2025

Version

5.3.0-SNAPSHOT

What happened?

The following query reveals a bug where an unexpected exception is raised in BindingBuilder.checkAdd: Attempt to reassign ?s from 1 to 1. The issue is in QueryIterLateral which just adds the lhs and rhs binding to the builder although they may have variables in common.

SELECT * {
    BIND(1 AS ?s)
    LATERAL {
        BIND(?s AS ?x)
        LATERAL {
            BIND(?s AS ?y)
        }
    }
}

My proposed solution in the PR is to add a check whether bindings are compatible on their common variables.

(Edit: I removed the report about the issues I encountered with var scoping so that this issue can be closed independently.)

Relevant output and stacktrace

No response

Are you interested in making a pull request?

Yes

@Aklakan Aklakan added the bug label Jan 7, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 7, 2025
…hecks on tables, enhanced QueryExec API for easier testing.
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 7, 2025
…hecks on tables, enhanced QueryExec API for easier testing.
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 8, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 8, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 8, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 8, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 8, 2025
@Aklakan Aklakan changed the title More Issues with Lateral Issue with Nested Laterals Jan 8, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 9, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 9, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 9, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 9, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 9, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 9, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 9, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 10, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 10, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 10, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 15, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 31, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Jan 31, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Feb 1, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Feb 1, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Feb 1, 2025
Aklakan added a commit to Aklakan/jena that referenced this issue Feb 1, 2025
afs pushed a commit that referenced this issue Feb 4, 2025
@afs afs closed this as completed in #2925 Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant