diff options
| author | Gunnar Aastrand Grimnes <gromgull@gmail.com> | 2017-05-14 16:15:43 +0200 |
|---|---|---|
| committer | Gunnar Aastrand Grimnes <gromgull@gmail.com> | 2017-05-14 16:16:54 +0200 |
| commit | 3696c10b94abd5d050836d46ddd9954d378f9831 (patch) | |
| tree | 61e02c6b71065cddd323501ce764538f643a5eb0 | |
| parent | 1cce4fd2dfcf1ed8159a070315608f4b226fb074 (diff) | |
| download | rdflib-3696c10b94abd5d050836d46ddd9954d378f9831.tar.gz | |
XMLResult: don't attempt to reraise exception with new message
there is no way to know if the original exception has a compatible
constructor
instead log a message and re-raise original exception.
| -rw-r--r-- | rdflib/plugins/sparql/results/xmlresults.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/rdflib/plugins/sparql/results/xmlresults.py b/rdflib/plugins/sparql/results/xmlresults.py index 20da0b2c..4d8c866b 100644 --- a/rdflib/plugins/sparql/results/xmlresults.py +++ b/rdflib/plugins/sparql/results/xmlresults.py @@ -1,3 +1,4 @@ +import logging from xml.sax.saxutils import XMLGenerator from xml.dom import XML_NAMESPACE @@ -19,6 +20,8 @@ from six import text_type SPARQL_XML_NAMESPACE = u'http://www.w3.org/2005/sparql-results#' RESULTS_NS_ET = '{%s}' % SPARQL_XML_NAMESPACE +log = logging.getLogger(__name__) + """A Parser for SPARQL results in XML: @@ -47,10 +50,8 @@ class XMLResult(Result): try: tree = etree.fromstring(xmlstring) except Exception as e: - try: - raise e.__class__("error parsing %r: %s" % (xmlstring, e)) - except: - raise e + log.exception("Error parsing XML results: %s"%xmlstring) + raise e boolean = tree.find(RESULTS_NS_ET + 'boolean') results = tree.find(RESULTS_NS_ET + 'results') |