Sent: Friday, January 28, 2011 9:08 PM
To: dev@thrift.apache.org
Subject: Re: [VOTE] Thrift 0.6.0 RC1
Mark,
this issue looks like
https://issues.apache.org/jira/browse/THRIFT-633, did you patch all
occurrences of misplaced @Override? In this case I guess THRIFT-633
could be closed too.
I currently
ira/browse/THRIFT-1051
> Committed revision 1064911.
>
> Cheers,
> mcslee
>
> -Original Message-
> From: Bryan Duxbury [mailto:br...@rapleaf.com]
> Sent: Friday, January 28, 2011 2:17 PM
> To: dev@thrift.apache.org
> Subject: Re: [VOTE] Thrift 0.6.0 RC1
>
>
om]
> Sent: Friday, January 28, 2011 2:17 PM
> To: dev@thrift.apache.org
> Subject: Re: [VOTE] Thrift 0.6.0 RC1
>
> Mind opening a ticket?
>
> On Fri, Jan 28, 2011 at 2:05 PM, Mark Slee wrote:
>
>> -1, just a minor issue. I'm +1 if we nuke the @Override cau
: dev@thrift.apache.org
Subject: Re: [VOTE] Thrift 0.6.0 RC1
Mind opening a ticket?
On Fri, Jan 28, 2011 at 2:05 PM, Mark Slee wrote:
> -1, just a minor issue. I'm +1 if we nuke the @Override causing problems
> below.
>
> The Java compile failed for me due to a bad @Override
> private static class TAsyncMethodCallTimeoutComparator implements
> Comparator {
>@Override
>public int compare(TAsyncMethodCall left, TAsyncMethodCall right) {
>
>
> -Original Message-
> From: Christian Lavoie [mailto:christian.lav...@gmail.com]
> Sen
n Lavoie [mailto:christian.lav...@gmail.com]
Sent: Thursday, January 27, 2011 7:55 PM
To: dev@thrift.apache.org
Subject: Re: [VOTE] Thrift 0.6.0 RC1
+1
Haskell works, and the whole thing compiles (and make checks) on Mac OS X 10.6.
On Thu, Jan 27, 2011 at 5:36 PM, Mathias Herberts
wr
+1
Haskell works, and the whole thing compiles (and make checks) on Mac OS X 10.6.
On Thu, Jan 27, 2011 at 5:36 PM, Mathias Herberts
wrote:
> +1
>
> My GWT patch was successfully updated for this RC.
>
--
Have fun, Christian
http://linkedin.christianlavoie.net
"I won't let you fall apart."
+1
My GWT patch was successfully updated for this RC.
Bryan,
I submitted four patches for the Node.js support last week that have
been merged to trunk. Without these patches, the implementation has a
lot of flaws. I think it would be nice to get this merged in to the
0.6.0 branch so that the first release of Node.js support doesn't go
out the door wi
