Yichun Zhang

Results 937 comments of Yichun Zhang

@rcosnita This has been on my TODO list and I'm glad you have looked into this! I'll check out your patch as soon as I can manage. Thank you!

Sorry, calling mysql procedures are currently not supported in ngx_drizzle because the underlying libdrizzle library does not support the CLIENT_MULTI_RESULTS flag yet :( But we'll surely work on it.

@ut0mt8 It's recommended to create github pull request to simplify code review and patch merging on our side. Thank you.

@ut0mt8 BTW, `receive("*a")` may be VERY expensive when the response is huge. Don't do that.

@gzliudan Seems like you call the API in a wrong way. You passed nil values to `combine()` instead of proper objects.

@splitice Thanks for your contribution! I'll look into this :)

I'm fine with `incr(key, value, init, expire)`. And I also want to have a separate `expire(time)` method :)

@splitice I understand that concern. Options tables are generally more expensive than passed-by-position arguments though :)

@splitice Personally I use `init` much more often ;) But I agree it can be more consistent with the `set` method's parameter order.