makeConcurrent dosen似乎在Gpars中工作

时间:2015-03-18 08:38:12

标签: multithreading groovy thread-safety threadpool gpars

以下程序应该返回具有空值的行列表。

import groovyx.gpars.GParsPool
import groovyx.gpars.ParallelEnhancer
import org.apache.commons.lang.RandomStringUtils
import groovyx.gprof.*
import static java.lang.Runtime.*;

def rows = []
def list = []

String charset = (('A'..'Z') + ('0'..'9')).join()
Integer length = 9

for (i in 0..1024) {
    rows[i] = RandomStringUtils.random(length, charset.toCharArray())
}

parallelResult = []
// Parallel execution

GParsPool.withPool(8) {
    parallelResult.makeConcurrent()
    rows.eachParallel {
        if (it[0] != null) {
            parallelResult.push(it)
        }
    }
    parallelResult.makeSequential()
}

// Sequential execution 
sequentialResult = []
rows.each {
    if (it[0] != null) {
        sequentialResult.push(it)
    }
}

assert parallelResult.size == sequentialResult.size

有时,结果并不相同。这可能是因为makeParallel存在一些问题。如果不是makeParallel,如何使用GPars创建并发列表?

1 个答案:

答案 0 :(得分:1)

我怀疑你在parallelResult上引入了竞争条件。 makeConcurrent()不会阻止集合上的竞争条件,它只会改变each(),collect()和此类方法的行为。

如果要使用累加器保留当前设计,则需要同步它或使用代理。但是,在我看来,您应该考虑使用更具功能性的方法。也许类似于“parallelResult = rows.findAllParallel {it [0]!= null}”