I'm pretty sure it is an oversight. Would you like to submit a pull request
to fix that?



On Tue, Aug 5, 2014 at 12:14 PM, Stephen Boesch <java...@gmail.com> wrote:

> Within its compute.close method, the JdbcRDD class has this interesting
> logic for closing jdbc connection:
>
>
>       try {
>         if (null != conn && ! stmt.isClosed()) conn.close()
>         logInfo("closed connection")
>       } catch {
>         case e: Exception => logWarning("Exception closing connection", e)
>       }
>
> Notice that the second check is on stmt  having been closed - not on the
> connection.
>
> I would wager this were not a simple oversight and there were some
> motivation for this logic- curious if anyone would be able to shed some
> light?   My particular interest is that I have written custom ORM's in jdbc
> since late 90's  and never did it this way.
>

Reply via email to