curator-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Drob (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (CURATOR-218) Race condition in client.blockUntilConnected()
Date Mon, 24 Aug 2015 12:25:45 GMT

     [ https://issues.apache.org/jira/browse/CURATOR-218?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Mike Drob resolved CURATOR-218.
-------------------------------
    Resolution: Fixed
      Assignee: Mike Drob  (was: Jordan Zimmerman)

Ran into this issue again when beating on the test suite over the weekend. Running with {{-Dsurefire-forkcount=8}}
created a lot of thread contention and other (existing) tests were failing in similar ways.

Pushed this and merged to 3.0

If there's any problems, I expect to see the unit tests shake this out in the next few days.

> Race condition in client.blockUntilConnected()
> ----------------------------------------------
>
>                 Key: CURATOR-218
>                 URL: https://issues.apache.org/jira/browse/CURATOR-218
>             Project: Apache Curator
>          Issue Type: Bug
>          Components: Recipes
>    Affects Versions: 2.7.0, 2.7.1, 2.8.0
>         Environment: host: AWS EC2 t2.medium type
> uname -a = Linux bamboo-agent 3.13.0-44-generic #73-Ubuntu SMP Tue Dec 16 00:22:43 UTC
2014 x86_64 x86_64 x86_64 GNU/Linux
> lsb_release -a = 
> Distributor ID:	Ubuntu
> Description:	Ubuntu 14.04.1 LTS
> Release:	14.04
> Codename:	trusty
> ZK version: is a curator TestingServer
> java: java 8
> maven: 3.2.5
>            Reporter: Samuel García Martínez
>            Assignee: Mike Drob
>             Fix For: 2.9.0
>
>         Attachments: failing_test.txt
>
>
> I'm implementing a Guava service relying on the startAsync feature. I'm using a listener
to wait for the INITIALIZED event to fire the guava service notifyStarted method, but sometimes
the listener never receives the event if the root path does not exist.
> I've created a repository with a failing test (looping over the failing test to force
it).
> repo with tests: https://github.com/samuelgmartinez/curator-treecache-tests.git
> failing test: mvn -Dtest=TreeCacheTest#testListenerNonExistentRootLoop test



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

Mime
View raw message