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

Reply via email to