yuxh/gearman-go
1
0
Fork 1
Code Issues Pull Requests Releases Wiki Activity

Remove client locking from pool.go

Browse Source 
client.go does do `client.Lock()` itself when it's needed. Doing the same in pool.go results in an dead lock.
This commit is contained in:
sewi-cpan 2019-11-08 13:06:14 +01:00 committed by GitHub
parent 81d00aa9ce
commit d40cb7e2b9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
client/pool.go
View File

@ -94,8 +94,6 @@ func (pool *Pool) Remove(addr string) {
func (pool *Pool) Do(funcname string, data []byte,
flag byte, h ResponseHandler) (addr, handle string, err error) {
client := pool.selectServer()
client.Lock()
defer client.Unlock()
handle, err = client.Do(funcname, data, flag, h)
addr = client.addr
return
@ -104,8 +102,6 @@ func (pool *Pool) Do(funcname string, data []byte,
func (pool *Pool) DoBg(funcname string, data []byte,
flag byte) (addr, handle string, err error) {
client := pool.selectServer()
client.Lock()
defer client.Unlock()
handle, err = client.DoBg(funcname, data, flag)
addr = client.addr
return
@ -115,8 +111,6 @@ func (pool *Pool) DoBg(funcname string, data []byte,
// !!!Not fully tested.!!!
func (pool *Pool) Status(addr, handle string) (status *Status, err error) {
if client, ok := pool.Clients[addr]; ok {
client.Lock()
defer client.Unlock()
status, err = client.Status(handle)
} else {
err = ErrNotFound
@ -136,8 +130,6 @@ func (pool *Pool) Echo(addr string, data []byte) (echo []byte, err error) {
return
}
}
client.Lock()
defer client.Unlock()
echo, err = client.Echo(data)
return
}