Repository navigation
docs(extensions): add SPARQL dialect - #659
ashlearnnext-star wants to merge 4 commits into
Conversation
|
Thank you for your first contribution to Apache Ossie (incubating)! A committer will review your PR. In the meantime, please ensure:
No ICLA is required to contribute. If you are elected as a committer in the future, you will need an Individual Contributor License Agreement (ICLA) on file with the ASF. Questions? Ask in the PR comments or join us on Slack. We appreciate your contribution! |
kayemkim
left a comment
kayemkim
left a comment
There was a problem hiding this comment.
SPARQL also needs to go into NON_SQL_DIALECTS in validation/validate.py. The validator keeps its own dialect tables, and since #612 a test pins them to dialects.md, so test_the_dialect_tables_match_the_registry fails on this branch. Without the entry, a model that uses SPARQL also gets "dialect 'SPARQL' is not supported by this validator", even though the registry lists it.
With that one line on top of current main, the validation and SDK tests pass. I've approved the workflow runs, so CI will show you the same thing.
|
Correction to my last line: these runs were queued on Oct 28, before #612 landed, so they tested the older main and came up green. Against current main the registry test does fail, so the |
|
creating a new one with teh updated changes |
Add SPARQL to the known dialects catalog.
SPARQL is a W3C-standard query language for RDF datasets and knowledge graphs. Documenting it in extensions/dialects.md provides a well-known dialect token for graph and semantic metadata use cases.
This is a documentation-only change.