search

hmsonline / storm-cassandra-cql

Storm Cassandra Bridge built on CQL
Apache License 2.0
43 stars 38 forks source link

Reduce synchronization on CqlClientFactory #54

Closed taikhuu closed 9 years ago

taikhuu commented 9 years ago

CqlClientFactory have synchronized eveything method which will greatly reduce parallelism unnecessary. Move synchronized to initalize methods.

bflad commented 9 years ago

I'll cut a release tomorrow morning unless there are any blocker comments. Thanks!

On Tue, Jul 28, 2015, 11:34 Bao Tran notifications@github.com wrote:

LGTM

— Reply to this email directly or view it on GitHub https://github.com/hmsonline/storm-cassandra-cql/pull/54#issuecomment-125655859 .

baontq commented 9 years ago

LGTM. Thanks @taikhuu @bflad

pabrahamsson commented 9 years ago

@baontq @taikhuu @bflad https://oss.sonatype.org/content/repositories/releases/com/hmsonline/storm-cassandra-cql/0.3.2/storm-cassandra-cql-0.3.2.jar