Skip to content

Commit

Permalink
MONDRIAN: Broadened the catch block in SqlStatement.execute() to incl…
Browse files Browse the repository at this point in the history
…ude all Exceptions, rather than just SQLException. This will make sure we clean up unused statements when RuntimeExceptions are thrown.

[git-p4: depot-paths = "//open/mondrian/": change = 13082]
  • Loading branch information
Matt Campbell authored and Matt Campbell committed Oct 2, 2009
1 parent e50ecc9 commit b324928
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/main/mondrian/rolap/SqlStatement.java
Expand Up @@ -144,7 +144,7 @@ public void execute() {
final long execMs = time - startTime;
Util.addDatabaseTime(execMs);
status = ", exec " + execMs + " ms";
} catch (SQLException e) {
} catch (Exception e) {
status = ", failed (" + e + ")";
try {
if (statement != null) {
Expand Down

0 comments on commit b324928

Please sign in to comment.