You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by "Daniel Sun (JIRA)" <ji...@apache.org> on 2017/09/15 16:46:00 UTC
[jira] [Closed] (GROOVY-8288) [Sql] withBatch fails when batchSize
== number of addBatch call
[ https://issues.apache.org/jira/browse/GROOVY-8288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Daniel Sun closed GROOVY-8288.
------------------------------
Resolution: Fixed
Assignee: Daniel Sun
Fix Version/s: 2.6.0-alpha-2
2.4.13
2.5.0-beta-2
3.0.0-alpha-1
Thanks for your patch!
> [Sql] withBatch fails when batchSize == number of addBatch call
> ---------------------------------------------------------------
>
> Key: GROOVY-8288
> URL: https://issues.apache.org/jira/browse/GROOVY-8288
> Project: Groovy
> Issue Type: Bug
> Components: SQL processing
> Affects Versions: 2.5.0-beta-1, 2.4.12
> Reporter: Antoine Kapps
> Assignee: Daniel Sun
> Fix For: 3.0.0-alpha-1, 2.5.0-beta-2, 2.4.13, 2.6.0-alpha-2
>
>
> {{Sql.withBatch(batchSize, ..)}} calls {{delegate.executeBatch()}} both when batchCount reaches batchSize, and at the end of the method (after the call to the "batch statement" closure).
> Thus, if the number of {{addBatch()}} calls in the closure is exactly the same as, or a multiple of {{batchSize}}, one too much call to {{executeBatch()}} is made : nothing has been added to the batch.
> Which, at least with HSQLDB 2.4.0, leads to a SQLException on that very last call.
> Test to reproduce (compliant with SqlBatchTest)
> {code:java}
> void testWithBatchSizeHavingSizeSameSizeAsStatements() {
> def numRows = sql.rows("SELECT * FROM PERSON").size()
> assert numRows == 3
> def myOthers = ['f4':'l4','f5':'l5','f6':'l6','f7':'l7']
> def result = sql.withBatch(myOthers.size(), "insert into PERSON (id, firstname, lastname) values (?, ?, ?)") { ps ->
> myOthers.eachWithIndex { k, v, index ->
> def id = index + numRows + 1
> ps.addBatch(id, k, v)
> }
> }
> assert result == [1] * myOthers.size()
> assert sql.rows("SELECT * FROM PERSON").size() == numRows + myOthers.size()
> // end result the same as if no batching was in place but logging should show:
> // FINE: Successfully executed batch with 4 command(s)
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)