Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/184#discussion_r85341210
--- Diff:
jena-arq/src/main/java/org/apache/jena/riot/tokens/TokenizerText.java ---
@@ -60,6 +61,26 @@
private boolean finished = false ;
private TokenChecker checker = null ;
+ // The code assumes that errors throw exception and so stop parsing.
+ private ErrorHandler errorHandler = new ErrorHandler() {
--- End diff --
Will do. It is stateless though a stateless object is not very expensive to
create!
The others are in ErrorHandlerFactory.
It maybe it becomes a parameter to the constructor.
Because letting dubious data through can lead to a lot of support load due
to downstream database problems, this PR just adds mechanism and does not
enable it. Hence it is keeping the default error handler in-file for now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
