-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10331/#review19126
-----------------------------------------------------------


Hi Vasanth, 
thank you very much for catching this one. We're clearly not properly closing 
all opened statements. It seems that we're creating the PreparedStatements 
inside the while loop which seems quite inefficient. What about creating the 
statements outside of the while loop and reuse them?

Jarcec

- Jarek Cecho


On April 7, 2013, 10:27 p.m., vasanthkumar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10331/
> -----------------------------------------------------------
> 
> (Updated April 7, 2013, 10:27 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Description
> -------
> 
> Show connection and job command hung when connection or job records size more 
> than 8.
> Problem while invoking prepareStatement method.
> 
> 
> This addresses bug sqoop-980.
>     https://issues.apache.org/jira/browse/sqoop-980
> 
> 
> Diffs
> -----
> 
>   
> repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
>  486635d 
> 
> Diff: https://reviews.apache.org/r/10331/diff/
> 
> 
> Testing
> -------
> 
> Done
> 
> 
> Thanks,
> 
> vasanthkumar
> 
>

Reply via email to