hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Purtell <apurt...@apache.org>
Subject Re: All ok w/ thrift2 going away in trunk/0.95?
Date Thu, 25 Apr 2013 18:23:50 GMT
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 <jxiang@cloudera.com> 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+YdVyO0aKDR1d4JCE65Q@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 <apurtell@apache.org
> >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 <lars.george@gmail.com>
> > 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 <stack@duboce.net> 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)

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message