Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
HBase, mail # dev - thrift vs thrift2


+
Jimmy Xiang 2012-12-06, 23:24
+
Stack 2012-12-07, 00:48
+
Jesse Yates 2012-12-07, 00:50
+
Andrew Purtell 2012-12-07, 03:21
+
Luke Lu 2012-12-07, 21:56
+
Stack 2012-12-07, 23:16
+
lars hofhansl 2012-12-08, 04:45
+
Andrew Purtell 2012-12-08, 05:06
+
lars hofhansl 2012-12-08, 05:27
Copy link to this message
-
Re: thrift vs thrift2
Andrew Purtell 2012-12-08, 05:42
Or it's easier to add more external thrift servers than tune up more
internal thrift server threads in a running RS? Speculation.

On Saturday, December 8, 2012, lars hofhansl wrote:

> Hmm... Interesting.Maybe the implementation is bad, but principally I
> don't see that. Why cache data twice when the in process thrift server
> could just access the data locally from the region server cache anyway?
>
> Anyway, either we should fix it or remove it. (just to state the obvious
> :) )
> I am not a fan of keeping non-functional code around in the hope that it
> might eventually get fixed.
>
>
> -- Lars
>
>
> ________________________________
>  From: Andrew Purtell <[EMAIL PROTECTED] <javascript:;>>
> To: "[EMAIL PROTECTED] <javascript:;>" <[EMAIL PROTECTED]<javascript:;>>;
> lars hofhansl <[EMAIL PROTECTED] <javascript:;>>
> Sent: Friday, December 7, 2012 9:06 PM
> Subject: Re: thrift vs thrift2
>
> IIRC the FB guys concluded the reverse, that it's better to provision
> Thrift endpoints as a tier separate from RegionServers, so there can be
> caching there, so the thrift servers in the RS don't bottleneck, etc.
>
> On Saturday, December 8, 2012, lars hofhansl wrote:
>
> > It's quite a useful experiment, though, standing up separate thrift
> > servers adds more complexity to a deployment.
> >
> > If it is really abandoned we should just rip it out, no? Otherwise I'd
> > think at least in principle that is useful (also considering potential
> > future enhancement that we do knowing the thrift server is in process
> with
> > the RegionServer).
> >
> > -- Lars
> >
> >
> >
> > ________________________________
> >  From: Stack <[EMAIL PROTECTED] <javascript:;> <javascript:;>>
> > To: HBase Dev List <[EMAIL PROTECTED] <javascript:;> <javascript:;>>
> > Sent: Friday, December 7, 2012 3:16 PM
> > Subject: Re: thrift vs thrift2
> >
> > On Fri, Dec 7, 2012 at 1:56 PM, Luke Lu <[EMAIL PROTECTED] <javascript:;><javascript:;>>
> > wrote:
> >
> > > The embeded HRegionThriftServer should be ported to thrift2 before we
> > > deprecate thrift1. FB's (and potentially others) c++ client requires
> > > HRegionThriftServer.
> > >
> >
> > HRTS is an abandoned experiment as I understand it so I closed
> HBASE-7302,
> > the issue that would perpetuate it in thrift2.
> > 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)
+
Ted Yu 2012-12-08, 05:12
+
Stack 2012-12-08, 07:31
+
Luke Lu 2012-12-08, 07:50
+
Jimmy Xiang 2012-12-08, 16:59
+
Andrew Purtell 2012-12-09, 02:15
+
Jake Farrell 2012-12-10, 20:36
+
Ted Yu 2012-12-10, 21:18
+
lars hofhansl 2012-12-08, 07:47
+
Ted Yu 2012-12-07, 22:01
+
Jimmy Xiang 2012-12-07, 22:20
+
Tim Sell 2012-12-07, 23:13
+
Stack 2012-12-07, 23:30
+
Andrew Purtell 2012-12-08, 01:45