Skip to content

Commit

Permalink
fix: keep wd: et al. prefixes for wikidata
Browse files Browse the repository at this point in the history
  • Loading branch information
rti committed Nov 13, 2024
1 parent 11fb8b2 commit 39795c6
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 3 deletions.
19 changes: 18 additions & 1 deletion build/wdqs/entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,24 @@ done

set -eu

export BLAZEGRAPH_OPTS="${BLAZEGRAPH_EXTRA_OPTS} -DwikibaseHost=${WIKIBASE_HOST} -DwikibaseConceptUri=${WIKIBASE_CONCEPT_URI}"
# Options provided to WDQS (blazegraph) when running as query service instance
#
# Note: We MUST not provide -DwikibaseHost=${WIKIBASE_HOST} here, otherwise
# WDQS would re-use the wd: et al. prefixes for the local wikibase instance.
# This is unintended, especially in the context of federation. wd: prefixes
# should remain in place for referencing wikidata. The local instance should
# choose its own prefixes, as described here:
# https://www.mediawiki.org/wiki/Wikibase/Wikibase.cloud/First_steps#View_your_data_using_the_Query_Service
# Some further thoughts on prefixes: https://phabricator.wikimedia.org/T335448
#
# In other words, WDQS does not know about the hostname of the wiki it gets
# its data from. Is is solely the task of the updater to feed data from the
# wiki into the WDQS instance.
export BLAZEGRAPH_OPTS="${BLAZEGRAPH_EXTRA_OPTS}"

# Options provided when running as wdqs-updater
#
# Here we provide -DwikibaseHost to reference the wiki to poll updates from.
export UPDATER_OPTS="-DwikibaseHost=${WIKIBASE_HOST} -DwikibaseMaxDaysBack=${WIKIBASE_MAX_DAYS_BACK}"

envsubst < /templates/mwservices.json > /wdqs/mwservices.json
Expand Down
2 changes: 0 additions & 2 deletions deploy/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,8 +98,6 @@ services:
hard: 32768
volumes:
- wdqs-data:/wdqs/data
environment:
WIKIBASE_CONCEPT_URI: https://${WIKIBASE_PUBLIC_HOST}
healthcheck:
test: curl --silent --fail localhost:9999/bigdata/namespace/wdq/sparql
interval: 10s
Expand Down

0 comments on commit 39795c6

Please sign in to comment.