hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Lars George <lars.geo...@gmail.com>
Subject Re: All ok w/ thrift2 going away in trunk/0.95?
Date Fri, 03 May 2013 21:04:05 GMT
Ah, good point. We do have a Thrift2 implementation, and I offered to take care of it. I am
just a bit unsure about where things are, do you know if all open issues are in JIRA? Or is
there something else more fundamentally wrong with it?

Thanks,
Lars

On May 3, 2013, at 6:06 PM, Andrew Purtell <apurtell@apache.org> wrote:

> Put up a patch for Thrift2 or this debate is academic.
> 
> On Friday, May 3, 2013, Lars George wrote:
> 
>> Hi Jimmy,
>> 
>> Inline...
>> 
>> On Apr 25, 2013, at 8:18 PM, Jimmy Xiang <jxiang@cloudera.com<javascript:;>>
>> 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
>> 
>> I am with Tim here (it was me that pushed Tim to complete this work in the
>> first place).
>> 
>>> 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.
>> 
>> That is independent if you ask me. We should have a proper Thrift one,
>> same with REST. Or do you want to toss out REST as well since we now have
>> PB RPCs?
>> 
>> I am willing to work and maintain Thrift2, I said that before. This thread
>> though got derailed in general wishful thinking, so could we please maybe
>> vote if we want Thrift and more especially Thrift2. Because we either throw
>> out Thrift in total for PB or maintain it for the time being.
>> 
>> Thoughts?
>> 
>> Cheers,
>> Lars
>> 
>>> 
>>> Thanks,
>>> Jimmy
>>> 
>>> 
>>> On Thu, Apr 25, 2013 at 11:10 AM, Andrew Purtell <apurtell@apache.org<javascript:;>
>>> 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<javascript:;>
>>> 
>>>> 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 <javascript:;>>
>> 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
View raw message