User: user57
Date: 01/08/16 15:23:30
Modified: src/main/org/jboss/ejb/plugins/jaws/jdbc
JDBCFinderCommand.java
Log:
o When execute() failes, log an error (instead of a debug) and throw
FinderException with more detail about the problem.
Revision Changes Path
1.15 +14 -11
jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCFinderCommand.java
Index: JDBCFinderCommand.java
===================================================================
RCS file:
/cvsroot/jboss/jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCFinderCommand.java,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -r1.14 -r1.15
--- JDBCFinderCommand.java 2001/08/12 10:46:22 1.14
+++ JDBCFinderCommand.java 2001/08/16 22:23:30 1.15
@@ -38,7 +38,7 @@
* @author <a href="mailto:[EMAIL PROTECTED]";>Joe Shevland</a>
* @author <a href="mailto:[EMAIL PROTECTED]";>Justin Forder</a>
* @author <a href="mailto:[EMAIL PROTECTED]";>Bill Burke</a>
- * @version $Revision: 1.14 $
+ * @version $Revision: 1.15 $
*
* <p><b>Revisions:</b>
*
@@ -81,20 +81,23 @@
}
- /** This method must be overridden to return the where clause used in
- * this query. This must start with the keyword 'WHERE' and include all
- * conditions needed to execute the query properly.
+ /**
+ * This method must be overridden to return the where clause used in
+ * this query. This must start with the keyword 'WHERE' and include all
+ * conditions needed to execute the query properly.
*/
public abstract String getWhereClause();
- /** This method must be ovverridden to return the full table list for
- * the query, including any join statements. This must start with the
- * keyword 'FROM' and include all tables needed to execute the query properly.
+ /**
+ * This method must be ovverridden to return the full table list for
+ * the query, including any join statements. This must start with the
+ * keyword 'FROM' and include all tables needed to execute the query properly.
*/
public abstract String getFromClause();
- /** This method must be ovverridded to return the full order by clause for
- * the query, including the 'ORDER BY' keyword.
+ /**
+ * This method must be ovverridded to return the full order by clause for
+ * the query, including the 'ORDER BY' keyword.
*/
public abstract String getOrderByClause();
@@ -113,8 +116,8 @@
result = new FinderResults(keys, null, null, null);
} catch (Exception e)
{
- if (log.isDebugEnabled()) log.debug("Exception", e);
- throw new FinderException("Find failed");
+ log.error("Failed to create finder results", e);
+ throw new FinderException("Find failed: " + e);
}
return result;
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development
