Skip to content

Additional SPARQL output formats for Data Distribution API - #521

Merged
brianjlowe merged 3 commits into
vivo-project:mainfrom
litvinovg:additional-sparql-output-formats
Jun 19, 2026
Merged

Additional SPARQL output formats for Data Distribution API#521
brianjlowe merged 3 commits into
vivo-project:mainfrom
litvinovg:additional-sparql-output-formats

Conversation

@litvinovg

Copy link
Copy Markdown
Member

Related to VIVO 4118
Related to VIVO 4119

  • Other Relevant Links (Mailing list discussion, related pull requests, etc.)

What does this pull request do?

Options to define output formats for SPARQL based Data distributors

What's new?

  • Added CSV output option for SPARQL Select Distributors
  • Added RDF/XML output option for SPARQL Construct Distributors
  • Added N3 output option for SPARQL Construct Distributors

How should this be tested?

  • Create Select from content Distributor, set result format to CSV, run distributor, check the output is CSV. Repeat the test with standard JSON result format.
  • Create Construct Query Graph builder and RDF Graph distributor, set RDF serialization option to RDF/XML, run distributor, check the output is RDF/XML. Repeat the test for N3 and TTL options.

Interested parties

@VIVO-project/vivo-committers

Reviewers' expertise

Candidates for reviewing this PR should have some of the following expertises:

  1. Java
  2. FreeMarker
  3. SPARQL

Reviewers' report template

General comment

A reviewer should provide here comments and suggestions for requested changes if any.

Testing

A reviewer should briefly describe here how it was tested

Code reviewing

A reviewer should briefly describe here which part was code reviewed

@litvinovg
litvinovg force-pushed the additional-sparql-output-formats branch from 66eff9b to ec412ea Compare May 4, 2026 13:52
chenejac
chenejac previously approved these changes May 8, 2026

@chenejac chenejac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@litvinovg looks good to me. There is one tiny comment, please take a look.

@litvinovg
litvinovg force-pushed the additional-sparql-output-formats branch from 8c7c8e5 to 82f2b0c Compare May 8, 2026 16:53
@litvinovg

Copy link
Copy Markdown
Member Author

Removed accidentally added commit to disable language filtering from this PR

@chenejac chenejac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@litvinovg well done

@brianjlowe brianjlowe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, and behaves as described.

@brianjlowe
brianjlowe merged commit 3ba5eca into vivo-project:main Jun 19, 2026
1 check passed
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.

3 participants