You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-commits@db.apache.org by cl...@apache.org on 2008/01/21 23:04:35 UTC
svn commit: r614029 -
/db/jdo/trunk/tck2-legacy/src/java/org/apache/jdo/tck/query/QueryTest.java
Author: clr
Date: Mon Jan 21 14:04:34 2008
New Revision: 614029
URL: http://svn.apache.org/viewvc?rev=614029&view=rev
Log:
JDO-570 Improve error handling; add query string to unexpected exception
Modified:
db/jdo/trunk/tck2-legacy/src/java/org/apache/jdo/tck/query/QueryTest.java
Modified: db/jdo/trunk/tck2-legacy/src/java/org/apache/jdo/tck/query/QueryTest.java
URL: http://svn.apache.org/viewvc/db/jdo/trunk/tck2-legacy/src/java/org/apache/jdo/tck/query/QueryTest.java?rev=614029&r1=614028&r2=614029&view=diff
==============================================================================
--- db/jdo/trunk/tck2-legacy/src/java/org/apache/jdo/tck/query/QueryTest.java (original)
+++ db/jdo/trunk/tck2-legacy/src/java/org/apache/jdo/tck/query/QueryTest.java Mon Jan 21 14:04:34 2008
@@ -28,6 +28,7 @@
import java.util.List;
import java.util.Map;
+import javax.jdo.JDOException;
import javax.jdo.JDOUserException;
import javax.jdo.PersistenceManager;
import javax.jdo.Query;
@@ -1303,8 +1304,14 @@
}
} catch (JDOUserException e) {
if (positive) {
- throw e;
- }
+ String msg = "JDOUserException thrown while executing query:\n" +
+ singleStringQuery;
+ throw new JDOException(msg, e);
+ }
+ } catch (JDOException e) {
+ String msg = "JDOException thrown while executing query:\n" +
+ singleStringQuery;
+ throw new JDOException(msg, e);
} finally {
if (tx.isActive()) {
tx.rollback();
@@ -1454,8 +1461,14 @@
tx.commit();
} catch (JDOUserException e) {
if (positive) {
- throw e;
- }
+ String msg = "JDOUserException thrown while executing query:\n" +
+ singleStringQuery;
+ throw new JDOException(msg, e);
+ }
+ } catch (JDOException e) {
+ String msg = "JDOException thrown while executing query:\n" +
+ singleStringQuery;
+ throw new JDOException(msg, e);
} finally {
if (tx.isActive()) {
tx.rollback();