Velocity
  1. Velocity
  2. VELOCITY-760

DataSourceResourceLoader doesn't close PreparedStatements

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6.2
    • Fix Version/s: 1.7, 2.x
    • Component/s: Engine
    • Labels:
      None

      Description

      DataSourceResourceLoader.java contains this method:

      private ResultSet readData(final Connection conn,
      final String columnNames,
      final String templateName) throws SQLException

      { PreparedStatement ps = conn.prepareStatement("SELECT " + columnNames + " FROM "+ tableName + " WHERE " + keyColumn + " = ?"); ps.setString(1, templateName); return ps.executeQuery(); }

      PreparedStatements created in this method never get closed, only the resultset returned may eventually be closed later which isn't sufficient for releasing all bound resources. In my project this statement leak lead to the oracle running out of open cursors (the infamous ORA-01000 error).

      1. velocity-760.patch
        3 kB
        Jarkko Viinamäki

        Activity

        Hide
        Jarkko Viinamäki added a comment -

        I looked at the source code of this class and to me it seems that all ResultSets and Connections are properly closed in finally-blocks. Are you sure this isn't an Oracle configuration issue?

        Show
        Jarkko Viinamäki added a comment - I looked at the source code of this class and to me it seems that all ResultSets and Connections are properly closed in finally-blocks. Are you sure this isn't an Oracle configuration issue?
        Hide
        Antonio Petrelli added a comment - - edited

        Closing result sets does not mean that prepared statements are closed.
        The opposite (closing prepared statements means closing result sets) is true.
        So essentially Jerome seems to be right.

        Show
        Antonio Petrelli added a comment - - edited Closing result sets does not mean that prepared statements are closed. The opposite (closing prepared statements means closing result sets) is true. So essentially Jerome seems to be right.
        Hide
        Malcolm Edgar added a comment -

        Agreed this is a bug, just found this recently as well. Note with Connection Pools this is can be a real problem.

        Show
        Malcolm Edgar added a comment - Agreed this is a bug, just found this recently as well. Note with Connection Pools this is can be a real problem.
        Hide
        Jarkko Viinamäki added a comment -

        This patch should fix it.

        Show
        Jarkko Viinamäki added a comment - This patch should fix it.

          People

          • Assignee:
            Unassigned
            Reporter:
            Jerome Waibel
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development