I would agree with removing it from the default build for now.
I only used thrift because that's what we were using for all of the
RPC at the time. I'd rather that we just settle on one RPC to rule
them all, and I will change the code accordingly.
On Aug 28, 2009, at 2:53 PM, Ryan, Michael P wrote:
I think it's okay for the locality server to depend on Thrift at
least until it becomes core functionality (the only thing we use it
for now is generating a list of work for benchmarks). When it
becomes generally useful, we can have a discussion about the merits
of Thrift. Maybe we can just remove it from the default build stuff
("make default")? Any thoughts, particularly from Jim?
- Michael
-----Original Message-----
From: Michael Stroucken
Sent: Thursday, August 27, 2009 10:43 PM
To: Ryan, Michael P; Julio Lopez
Subject: tashi and thrift
Hi,
I'd like to make a commit of updated documentation and code that
removes
several references on thrift. However, thrift is still imported in
several places, including the locality server. What is your take on
this?
Greetings,
Michael.
