-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9965/
-----------------------------------------------------------
Review request for oodt and Chris Mattmann.
Description
-------
The loadScript method uses a BufferedReader resource but doesn't close it after
use. Since the method declares 'throws IOException', we can use try...finally
and close the reader in the finally block. Additionally, it looks like the
method deals with files and strings only (no SQL stuff), so we can remove the
'throws SQLException' from the method statement.
This addresses bug OODT-576.
https://issues.apache.org/jira/browse/OODT-576
Diffs
-----
/trunk/commons/src/main/java/org/apache/oodt/commons/database/SqlScript.java
1457043
Diff: https://reviews.apache.org/r/9965/diff/
Testing
-------
I ran the unit tests for the commons package. I also ran the
TestWorkflowDataSourceRepository tests from the workflow package, as these
tests use a setUp method that makes a call to the loadScript method in
SqlScript.
Thanks,
Ross Laidlaw
