Go Commons Pool发布以及Golang多线程编程问题总结

jolestar · · 2020 次点击 · · 开始浏览    
这是一个创建于 的文章,其中的信息可能已经有所发展或是发生改变。

趁着元旦放假,整理了一下最近学习Golang时,『翻译』的一个Golang的通用对象池,放到 github Go Commons Pool开源出来。之所以叫做『翻译』,是因为这个库的核心算法以及逻辑都是基于 Apache Commons Pool 的,只是把原来的Java『翻译』成了Golang。

前一段时间阅读kubernetes源码的时候,整体上学习了下Golang,但语言这种东西,学了不用,几个星期就忘差不多了。一次Golang实践群里聊天,有人问到Golang是否有通用的对象池,搜索了下,貌似没有比较完备的。当前Golang的pool有以下解决方案:

  1. sync.Pool
    sync.Pool 使用很简单,只需要传递一个创建对象的func即可。

     var objPool = sync.Pool{
     New: func() interface{} {
     return NewObject()}}
     p := objPool.Get().(*Object)
    

    但sync.Pool只解决对象复用的问题,pool中的对象生命周期是两次gc之间,gc后pool中的对象会被回收,使用方不能控制对象的生命周期,所以不适合用在连接池等场景。

  2. 通过container/list来实现自定义的pool,比如redigo 就使用这种办法。但这些自定义的pool大多都不是通用的,功能也不完备。比如redigo当前没有获取连接池的超时机制,参看这个issue Blocking with timeout when Get PooledConn

而Java中的commons pool,功能比较完备,算法和逻辑也经过验证,使用也比较广泛,所以就直接『翻译』过来,顺便练习Golang的语法。

作为一个通用的对象池,需要包含以下主要功能:

  1. 对象的生命周期可以精确控制 Pool提供机制允许使用方自定义对象的创建/销毁/校验逻辑
  2. 对象的存活数量可以精确控制 Pool提供设置存活数量以及时长的配置
  3. 获取对象有超时机制避免死锁,方便使用方实现failover 以前也遇到过许多线上故障,就是因为连接池的设置或者实现机制有缺陷导致的。

Apache Commons Pool的核心是基于LinkedBlockingDeque,idle对象都放在deque中。之所以是deque,而不是queue,是因为它支持LIFO(last in, first out) /FIFO(first in, first out) 两种策略获取对象。然后有个包含所有对象的Map,key是用户自定义对象,value是PooledObject,用于校验Return Object的合法性,后台定时abandoned时遍历,计算活跃对象数等。超时是通过Java锁的wait timeout机制实现的。

下面总结下将Java翻译成Golang的时候遇到的多线程问题

递归锁或者叫可重入锁(Recursive Lock)

Java中的synchronized关键词以及LinkedBlockingDequeu中用到的ReentrantLock,都是可重入的。而Golang中的sync.Mutex是不可重入的。表现出来就是:

ReentrantLock lock;

public void a(){
    lock.lock();
    //do some thing
    lock.unlock();
}

public void b(){
    lock.lock();
    //do some thing
    lock.unlock();
}

public void all(){
    lock.lock();
    //do some thing
    a();
    //do some thing
    b();
    //do some thing
    lock.unlock();
}

上例all方法中嵌套调用a方法,虽然调用a方法的时候也需要锁,但因为all已经申请锁,并且该锁可重入,所以不会导致死锁。而同样的代码在Golang中是会导致死锁的:

var lock sync.Mutex

func a() {
    lock.Lock()
    //do some thing
    lock.Unlock()
}

func b() {
    lock.Lock()
    //do some thing
    lock.Unlock()
}

func all() {
    lock.Lock()
    //do some thing
    a()
    //do some thing
    b()
    //do some thing
    lock.Unlock()
}

只能重构为下面这样的(命名不规范请忽略,只是demo)

var lock sync.Mutex

func a() {
    lock.Lock()
    a1()
    lock.Unlock()
}

func a1() {
    //do some thing
}

func b() {
    lock.Lock()
    b1()
    lock.Unlock()
}

func b1() {
    //do some thing
}

func all() {
    lock.Lock()
    //do some thing
    a1()
    //do some thing
    b1()
    //do some thing
    lock.Unlock()
}

Golang的核心开发者认为可重入锁是不好的设计,所以不提供,参看Recursive (aka reentrant) mutexes are a bad idea。于是我们使用锁的时候就需要多注意嵌套以及递归调用。

锁等待超时机制

Golang的 sync.Cond 只有Wait,没有如Java中的Condition的超时等待方法await(long time, TimeUnit unit)。这样就没法实现LinkBlockingDeque的 pollFirst(long timeout, TimeUnit unit) 这样的方法。有人提了issue,但被拒绝了 sync: add WaitTimeout method to Cond。 所以只能通过channel的机制模拟了一个超时等待的Cond。完整源码参看 go-commons-pool/concurrent/cond.go

type TimeoutCond struct {
    L      sync.Locker
    signal chan int
}

func NewTimeoutCond(l sync.Locker) *TimeoutCond {
    cond := TimeoutCond{L: l, signal: make(chan int, 0)}
    return &cond
}

/**
return remain wait time, and is interrupt
*/
func (this *TimeoutCond) WaitWithTimeout(timeout time.Duration) (time.Duration, bool) {
    //wait should unlock mutex,  if not will cause deadlock
    this.L.Unlock()
    defer this.L.Lock()
    begin := time.Now().Nanosecond()
    select {
    case _, ok := <-this.signal:
        end := time.Now().Nanosecond()
        return time.Duration(end - begin), !ok
    case <-time.After(timeout):
        return 0, false
    }
}

Map机制的问题

这个问题严格的说不属于多线程的问题。虽然Golang的map不是线程安全的,但通过mutex封装一下也很容易实现。关键问题在于我们前面提到的,pool中用于维护全部对象的map,key是用户自定义对象,value是PooledObject。而Golang对map的key的约束是:go-spec#Map_types

The comparison operators == and != must be fully defined for operands of the key type; thus the key type must not be a function, map, or slice. If the key type is an interface type, these comparison operators must be defined for the dynamic key values; failure will cause a run-time panic.

也就是说key中不能包含不可比较的值,比如 slice, map, and function。而我们的key是用户自定义的对象,没办法进行约束。于是借鉴Java的IdentityHashMap的思路,将key转换成对象的指针地址,实际上map中保存的是key对象的指针地址。

type SyncIdentityMap struct {
    sync.RWMutex
    m map[uintptr]interface{}
}

func (this *SyncIdentityMap) Get(key interface{}) interface{} {
    this.RLock()
    keyPtr := genKey(key)
    value := this.m[keyPtr]
    this.RUnlock()
    return value
}

func genKey(key interface{}) uintptr {
    keyValue := reflect.ValueOf(key)
    return keyValue.Pointer()
}

同时,这样做的缺点是Pool中存的对象必须是指针,不能是值对象。比如string,int等对象是不能保存到Pool中的。

其他的关于多线程的题外话

Golang的test -race 参数非常好用,通过这个参数,发现了几个data race的bug,参看commit fix data race test error

Go Commons Pool后续工作

  1. 继续完善测试用例,测试用例当前已经完成了大约一半多,覆盖率88%。『翻译』的时候,主体代码相对来说写起来很快,但测试用例就比较麻烦多了,多线程情况下调试也比较复杂。一般基础库的测试用例代码是核心逻辑代码的2-3倍。
  2. 做下benchmark。核心算法上应该没啥问题,都是进过验证的。但用channel模拟timeout的机制上可能有瓶颈。这块要考虑timer的复用机制。参看 Terry-Mao/goim
  3. 上两项完成了,就可以准备发布个正式版本,可以通过这个pool改进下redigo。

有疑问加站长微信联系(非本文作者)

本文来自:午夜咖啡

感谢作者:jolestar

查看原文:Go Commons Pool发布以及Golang多线程编程问题总结

入群交流(和以上内容无关):加入Go大咖交流群,或添加微信:liuxiaoyan-s 备注:入群;或加QQ群:692541889

2020 次点击  
加入收藏 微博
暂无回复
添加一条新回复 (您需要 登录 后才能回复 没有账号 ?)
  • 请尽量让自己的回复能够对别人有帮助
  • 支持 Markdown 格式, **粗体**、~~删除线~~、`单行代码`
  • 支持 @ 本站用户;支持表情(输入 : 提示),见 Emoji cheat sheet
  • 图片支持拖拽、截图粘贴等方式上传