One Gearmand restart leads to worker to lose connection to all servers

Bug #655812 reported by Miguel Simões
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gearman Java
In Progress
High
Eric Lambert

Bug Description

When you are working with more then one server associated with the GearmanWorker, if one of the servers terminates the connection, the Worker tries to shutdown it self instead making it loose the connection to all servers instead of only termating the connection on the server that lost the connection.

Revision history for this message
Miguel Simões (miguel-simoes) wrote :

Terrible english there ^^

When you add more then one server to the GearmanWorker, when one of the gearmand shutsdown, the GearmanWorker kills himself and looses the connection to all servers registered, instead it should just shutdown the connection to the gearmand that lost the connection.

PS: this is related with the issue of restoring the connection to a gearmand that has, for example, a restart and lost connection for a brief period.

Revision history for this message
Eric Lambert (elambert) wrote :

Hi Miguel:

Your description makes sense to me. Will take a look at this.

Thanks for filling the bug.

Cheers!

Eric

Changed in gearman-java:
assignee: nobody → Eric Lambert (elambert)
importance: Undecided → High
Revision history for this message
Eric Lambert (elambert) wrote :

It looks like this is cause by the sess.getSelectionKey().interestOps(interestOps) in GearmanWorkerImpl.run() throwing an unchecked exception (IllegalStateException). There are some other bugs relating to how the worker deals with dropped exceptions which I plan to fix in 0.05, so will address this issue as well.

Changed in gearman-java:
milestone: none → 0.05
status: New → Confirmed
Eric Lambert (elambert)
Changed in gearman-java:
status: Confirmed → In Progress
Matt Fischer (mfisch)
summary: - One Gearmand restart leads to worker to loose connection to all servers
+ One Gearmand restart leads to worker to lose connection to all servers
Eric Lambert (elambert)
Changed in gearman-java:
milestone: 0.05 → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.