-
Notifications
You must be signed in to change notification settings - Fork 8
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
Gh 184 introduce wikibase service clause #185
Conversation
There is still a bug, the query optimizer is reordering the queries. The SERVICE clause should be evaluated last in order to have the bindingSet filled. This is currently not always the case |
...qs-backend/src/main/java/com/the_qa_company/qendpoint/federation/ServiceClauseOptimizer.java
Outdated
Show resolved
Hide resolved
hdt-qs-backend/src/main/java/com/the_qa_company/qendpoint/federation/WikibaseLabelService.java
Outdated
Show resolved
Hide resolved
hdt-qs-backend/src/main/java/com/the_qa_company/qendpoint/federation/WikibaseLabelService.java
Outdated
Show resolved
Hide resolved
hdt-qs-backend/src/main/java/com/the_qa_company/qendpoint/federation/WikibaseLabelService.java
Outdated
Show resolved
Hide resolved
...ckend/src/main/java/com/the_qa_company/qendpoint/federation/SPARQLServiceResolverCustom.java
Outdated
Show resolved
Hide resolved
hdt-qs-backend/src/main/java/com/the_qa_company/qendpoint/federation/WikibaseLabelService.java
Outdated
Show resolved
Hide resolved
wow this interface is so complicated! I made all the changes already this morning directly after we discussed .... you did not get any notification right? @ate47 Is there something pending? |
Fine for me, can you squash your commits before the merge? |
Apply formatter Windows disclaimer Preparing for release 1.7.1 fix docker CI fix ci #173 GH-167 sync writer (#176) GH-167 sync writer GH-181 Remove shutdown and add copy test (#182) First complete version of Wikibase query service for languages Running formatter Rename class to more meaningfull name Rename class to more meaningfull name Don't return null but throw error at the end of the iterator
…thub.com/the-qa-company/qEndpoint into GH-184-introduce-Wikibase-service-clause
Description of this pull request:
This is a first draft of the Wikibase language SERVICE functionality described here:
https://en.wikibooks.org/wiki/SPARQL/SERVICE_-_Label
For now only the Automatic Label SERVICE is implemented. Also some more testing is needed
Please check all the lines before posting the pull request:
mvn formatter:format
on the backend,npm run format
on the frontend) before posting my pull request,mvn formatter:validate
to validate the formatting on the backend,npm run validate
on the frontend