On Mar 11, 2014, at 8:12 PM, yun...@apache.org wrote: > TS-2612: Indroduce TSHttpConnectWithProtoStack() API > > 1) Firstly, add a bitmask, *proto_stack*, in NetVConnection/HttpSM, > and set it properly. > > 2) For some plugins that using TSHttpConnect() API to do request, > the Logging module can't know what protocol stack is used, so I > add a new API: > > TSHttpConnectWithProtoStack(struct sockaddr const* addr, > TSClientProtoStack proto_stack); > > After introducing TSHttpConnectWithProtoStack() API, TSHttpConnect() API > would be a special case of it: > > TSVConn > TSHttpConnect(sockaddr const* addr) > { > return TSHttpConnectWithProtoStack(addr, (1u << TS_PROTO_HTTP)); > } > > Signed-off-by: Yunkai Zhang <qiushu....@taobao.com>
This needs API review since the final form of the API was not reviewed on dev@. I'll try to review this next week. Everyone else who reviewed the original proposal should also review :) J