PB gateway is an interesting proposal. I always thought of just making our PB RPC an api, but we are not there yet. If this will help us getting close, I will be a big +1.
Enis On Thu, Apr 25, 2013 at 11:23 AM, Andrew Purtell <[email protected]>wrote: > That's an interesting idea, although maybe not what you meant: If we had a > PB only API gateway (e.g. no "PBUF" markers, no Writables anywhere, etc.) > then this might be a way to evolve RPC to that state informed by experience > with said gateway. > > > On Thu, Apr 25, 2013 at 11:18 AM, Jimmy Xiang <[email protected]> wrote: > > > At first, I am +1 for removing it. We had a similar discussion before, > and > > didn't pull the plug because of Tim's comment: > > > > > > > http://mail-archives.apache.org/mod_mbox/hbase-dev/201212.mbox/%3ccae9mebbh7v1phsbegtepbxg1h5drbs+ydvyo0akdr1d4jce...@mail.gmail.com%3E > > > > To me, instead of complete and maintain Thrift2, it will be much better > to > > come up a new one since we are on PB now. > > > > Thanks, > > Jimmy > > > > > > On Thu, Apr 25, 2013 at 11:10 AM, Andrew Purtell <[email protected] > > >wrote: > > > > > I'm glad someone has stepped forward to be an active responsive > > maintainer > > > of this piece of code. Maintenance is one issue, actual usage is > another. > > > Does anyone actually use this? What is the plan for Thrift? Do we > > continue > > > with both interfaces through one or more subsequent versions? > > > > > > > > > On Wed, Apr 24, 2013 at 10:45 PM, Lars George <[email protected]> > > > wrote: > > > > > > > I am -1 to remove, took a long time to get it in there and should > > > > deprecate Thrift v1 - or else we are in the same mess as mapred and > > > > mapreduce is. We once replaced the entire client API and now we can't > > do > > > > this for Thrift? > > > > > > > > I am happy to work on v2 and fix or maintain it. It should be the way > > > > forward methinks. > > > > > > > > Lars > > > > > > > > On Apr 24, 2013, at 21:53, Stack <[email protected]> wrote: > > > > > > > > > Thrift2 was supposed to be the future -- an API like the native > java > > > API > > > > -- > > > > > but it never got the support needed to make it a superset of > thrift1. > > > > > Meantime folks are running thrift1 in production and patching it as > > > they > > > > > need extra calls and fixes. > > > > > > > > > > While we have two thrifts, thrift1 and thrift2, the latter of which > > we > > > > are > > > > > afraid to recommend because we do not know of any prod install, we > > sew > > > > > confusion ("Which should I use?", "Doc only talks about > 'thrift'?"). > > > > > > > > > > HBASE-8184 is a patch to remove thrift2 until it gets a sponsor who > > can > > > > > move it beyond thrift1 . I already have a +1 to remove. Will let > > the > > > > > issue stew a day or two in case objection to thrift2 going away. > > > > > > > > > > St.Ack > > > > > > > > > > > > > > > > -- > > > Best regards, > > > > > > - Andy > > > > > > Problems worthy of attack prove their worth by hitting back. - Piet > Hein > > > (via Tom White) > > > > > > > > > -- > Best regards, > > - Andy > > Problems worthy of attack prove their worth by hitting back. - Piet Hein > (via Tom White) >
