Skip to content

Comments

Add a BIND at the end of the query as a workaround and remove a dupli…#2524

Open
BenjaminHofstetter wants to merge 2 commits intomainfrom
LINDASSUP-186_distinct_problem
Open

Add a BIND at the end of the query as a workaround and remove a dupli…#2524
BenjaminHofstetter wants to merge 2 commits intomainfrom
LINDASSUP-186_distinct_problem

Conversation

@BenjaminHofstetter
Copy link

Issue from https://cognizone.atlassian.net/servicedesk/customer/portal/1/LINDASSUP-186

I create an issue for the DISTICNT problem with GraphDB
https://cognizone.atlassian.net/servicedesk/customer/portal/1/LINDASSUP-191
Closes #

This PR adds a BIND to a query to make it work with GraphDB.


@vercel
Copy link
Contributor

vercel bot commented Feb 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
visualization-tool Ready Ready Preview, Comment Feb 20, 2026 0:26am

Request Review

@BenjaminHofstetter BenjaminHofstetter removed the request for review from tpluscode February 20, 2026 12:17
@BenjaminHofstetter BenjaminHofstetter marked this pull request as draft February 20, 2026 12:17
@BenjaminHofstetter BenjaminHofstetter marked this pull request as ready for review February 20, 2026 12:21
BIND(str(coalesce(str(?symbol), str(?ucumCode), str(?expression), str(?rdfsLabel), "?")) AS ?l)

FILTER ( lang(?rdfsLabel) = "${locale}" || lang(?rdfsLabel) = "en" || datatype(?rdfsLabel) = ${ns.xsd.string} )
BIND(xsd:string(?l) AS ?label)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. why the additional xsd:string cast?
  2. you can combine both BIND anyway

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The xsd:string cast is a workaround for a graphDB issue. It ensures that all types and language tags are removed or the same, making the working distinct.
Since it’s a workaround, I don’t want to combine it with the other BIND. Perhaps it can be removed later.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, let's make a comment so that is clear in the future, please

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