[THRIFT-3112] [Java] AsyncMethodCallback should be typed in generated AsyncIface

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.9.2, 0.9.3
• Fix Version/s: 0.10.0
• Component/s: Java - Compiler, Java - Library
• Assignee: John Sirois
• Reporter: Sergei Egorov
• Created: 25/Apr/15 11:32
• Updated: 05/Apr/17 15:39
• Resolved: 18/Mar/16 15:35

Description

AsyncMethodCallback is generic, but current Java code generator is not adding type info to it, and instead of:

public interface AsyncIface {
    public void ping(org.apache.thrift.async.AsyncMethodCallback<String> resultHandler) throws org.apache.thrift.TException;
 }

we have:

public interface AsyncIface {
    public void ping(org.apache.thrift.async.AsyncMethodCallback resultHandler) throws org.apache.thrift.TException;
 }

Issue Links

• blocks: THRIFT-3583 Add support for generating immutable java stubs for structs and unions
• depends upon: THRIFT-1972 Add support for async processors
• is related to: THRIFT-2157 generated code would cause ClassCastException
• relates to: THRIFT-3312 Why java async client not use Future<V> as result?
• links to: GitHub Pull Request #500