The embeded HRegionThriftServer should be ported to thrift2 before we deprecate thrift1. FB's (and potentially others) c++ client requires HRegionThriftServer.
__Luke On Thu, Dec 6, 2012 at 7:21 PM, Andrew Purtell <[email protected]> wrote: > +1 for deprecation. Will bring any users out of the woodwork who might not > see this discussion at least. > > On Friday, December 7, 2012, Jesse Yates wrote: > > > According to the thrift2 docs, it is the 'right' way to access HBase - > the > > thrift package is based on an older HBase interface that no longer > exists. > > Seems like a reasonable case for deprecation in the next release and > > dropping it all together after. > > ------------------- > > Jesse Yates > > @jesse_yates > > jyates.github.com > > > > > > On Thu, Dec 6, 2012 at 4:48 PM, Stack <[email protected] <javascript:;>> > > wrote: > > > > > On Thu, Dec 6, 2012 at 3:24 PM, Jimmy Xiang <[email protected] > <javascript:;>> > > wrote: > > > > > > > Since 0.96 is a singularity, is it a time to deprecate thrift > finally? > > > > > > > > Or we can use thrift2 by default, and keep thrift around more? > > > > > > > > What's the pros and cons to switch to thrift2? > > > > > > > > > > Interesting notion. Could move thrift2 to thrift altogether. Would > need > > > to make sure thrift2 has all from thrift1. > > > St.Ack > > > > > > > > -- > Best regards, > > - Andy > > Problems worthy of attack prove their worth by hitting back. - Piet Hein > (via Tom White) >
