[ 
https://issues.apache.org/jira/browse/GROOVY-7673?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15005588#comment-15005588
 ] 

acefael commented on GROOVY-7673:
---------------------------------

to reproduce a code block like below could be used:

{code}
def sql = new groovy.sql.Sql( dataSource() )
def vals = []
200000.times { i -> vals.add( { i -> sql.execute( [ key: i ] , 'insert into 
tbl(col)values(:key)' ) }.curry(i) }
def pool = java.util.concurrent.Executors.newFixedThreadPool( 10 )
pool.invokeAll(vals)
pool.shutdown()
while(true){pool.awaitTermination(5,TimeUnit.SECONDS)}
{code}


> groovy.sql.Sql race condition?
> ------------------------------
>
>                 Key: GROOVY-7673
>                 URL: https://issues.apache.org/jira/browse/GROOVY-7673
>             Project: Groovy
>          Issue Type: Question
>    Affects Versions: 2.4.5
>            Reporter: acefael
>
> I am using groovy.sql.Sql with a commons dbcp2 BasicDataSource and am 
> encountering a stacktrace as in \[2\] below.  I do create many identical 
> closures with different data, and I pass them all at once to 
> ExecutorService.invokeAll.  The closures are all using the same Sql instance.
> I believe this is a race condition because it happens only when the database 
> is over a network, and not when the database is local.
> To work around I can set Sql.cacheNamedQueries = false.  Problem is reliably 
> gone in this case.  So I guess the culprit is in \[1\], with my sql already 
> being cached in namedParamSqlCache, but not yet in namedParamIndexPropCache?
> \[1\] the line in the source code
> https://github.com/apache/incubator-groovy/blob/GROOVY_2_4_X/subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java#L4407
> \[2\] the stacktrace
> {noformat}
> > java.lang.NullPointerException
> >       at java.util.ArrayList.<init>(ArrayList.java:168) ~[?:1.8.0]
> >       at groovy.sql.Sql.buildSqlWithIndexedProps(Sql.java:4422) 
> > ~[groovy-all-2.4.5.jar:2.4.5]
> >       at groovy.sql.Sql.checkForNamedParams(Sql.java:4369) 
> > ~[groovy-all-2.4.5.jar:2.4.5]
> >       at groovy.sql.Sql.getPreparedStatement(Sql.java:4360) 
> > ~[groovy-all-2.4.5.jar:2.4.5]
> >       at groovy.sql.Sql.getPreparedStatement(Sql.java:4439) 
> > ~[groovy-all-2.4.5.jar:2.4.5]
> >       at groovy.sql.Sql.execute(Sql.java:2366) ~[groovy-all-2.4.5.jar:2.4.5]
> >       at groovy.sql.Sql.execute(Sql.java:2438) ~[groovy-all-2.4.5.jar:2.4.5]
> >       at groovy.sql.Sql$execute$1.call(Unknown Source) ~[?:?]
> >       at es.acefael.BaseMigration.execute(BaseMigration.groovy:2017) 
> > ~[es-acefael-dostuff-SNAPSHOT.jar:?]
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to