Re: THRIFT-3854 (Java) - Please Review

2017-01-26 Thread John Sirois
I noted a few minor things, but not bugs - looks good to me. On Thu, Jan 26, 2017 at 1:38 PM, Jim King wrote: > I’d like someone more familiar with the Java runtime to review the change > before I merge it. It passed CI and has a nice unit test. > >

THRIFT-3854 (Java) - Please Review

2017-01-26 Thread Jim King
I'd like someone more familiar with the Java runtime to review the change before I merge it. It passed CI and has a nice unit test. https://github.com/apache/thrift/pull/1081 [Description: Description: simplivity-lg-xsmall] James E. King, III Architect 8 Technology Drive, 2nd Floor Westborough,