chenejac / VIVOTestMigration

0 stars 0 forks source link

VIVO-1943: StackOverflowException during parsing of SPARQL Update API request #1826

Closed chenejac closed 3 years ago

chenejac commented 3 years ago

Brian Lowe (Migrated from VIVO-1943) said:

INSERT DATA { } requests with more than roughly 5K triples can/will throw StackOverflowExceptions due to an issue with the underlying Jena libraries discussed here:

[https://mail-archives.apache.org/mod_mbox/jena-users/201601.mbox/%3C568BA9FB.5090709@apache.org%3E]

The workaround mentioned (to use the "ARQ" parser instead of the "SPARQL" parser) is what I have been using for some time, and does indeed avoid the problem.  It is not immediately clear whether newer versions of Jena still exhibit this issue.

In edu.cornell.mannlib.vitro.webapp.controller.api.SparqlUpdateApiController, I changed the line around line 94 to:

return UpdateFactory.create(update, Syntax.syntaxARQ);

 

chenejac commented 3 years ago

Andrew Woods said:

[~accountid:557058:3f53c733-6ac2-4e92-9f12-e34e297283a3]: I don't necessarily expect this PR to fix the issue, but it would be good if you could verify: https://github.com/vivo-project/Vitro/pull/196

chenejac commented 3 years ago

Andrew Woods said:

Resolved with: https://github.com/vivo-project/Vitro/commit/936305be2a6a5bbb63381561e241997d9cbcc261