I'm not an ActionScript expert, but I have used it quite a bit. That being said, I've been using this patch (and the private branch that preceded it) for at least a month now and it seems to work great. We are only using the serialization support, not the server support, but we've been using it extensively.
So, +1 from me, if that helps :) Jason On Mon, Mar 8, 2010 at 4:33 PM, David Reiss (JIRA) <[email protected]> wrote: > > [ > https://issues.apache.org/jira/browse/THRIFT-518?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12842856#action_12842856 > ] > > David Reiss commented on THRIFT-518: > ------------------------------------ > > This looks reasonable, though I didn't review it super-thoroughly since I'm > not familiar with ActionScript. I have a few small complaints: > > - gitignore changes should be in a separate diff. > - Commented-out code should be deleted. > - indent is of fin a few places in the generator. > > I'd prefer if someone else who knows ActionScript could review this better. > If not, I guess we can commit. > > I haven't reviewed the full-duplex-http design yet, so don't be surprised if > I ask for changes to that after-the-fact. > > >> as3/flash/flex generator >> ------------------------ >> >> Key: THRIFT-518 >> URL: https://issues.apache.org/jira/browse/THRIFT-518 >> Project: Thrift >> Issue Type: New Feature >> Reporter: Dave Lerman >> Attachments: as3.patch, TFullDuplexHttpClient.as, thrift_as3.diff, >> thrift_as3.diff >> >> >> There's been various mailing list discussions about ActionScript 3 support, >> but I didn't see an associated JIRA so I thought I'd create one. >> The goal would be to allow a Flash or Flex project to call a Thrift service >> as an alternative to Flash's built-in web services and RPC implementations. >> A developer might want to use Thrift instead of SOAP, REST or AMF for it's >> code generation, strong typing, or for interoperability with existing >> Thrift-based services. >> The Flash code would look something like: >> {code} >> public function testFunction() { >> var client:Service = new ServiceImpl(new TBinaryProtocol( >> new THttpClient(new URLRequest("http://service.com"))); >> client.ping("hello world", handlePingResponse); >> } >> private function handlePingResponse(response:String):void { >> trace("RESPONSE: " + response); >> } >> {code} >> where Service is the generated Flash interface, ServiceImpl is the generated >> client which implements Service, and THttpClient is an implementation of >> TTransport. >> Note that Flash is a single-threaded environment so the call is necessarily >> asynchronous. >> The attached patch is a first-pass at an implementation. It's basically a >> line-for-line partial port of the java lib and generator. The lib contains >> a single protocol (TBinaryProtocol) and a single transport (THttpClient), >> along with a generator which generates the interface and client >> implementation (server implementation is skipped since this seems unlikely >> to be useful). It still needs some work -- it's untested except for the >> specific thrift services we use internally, and needs documentation and >> cleanup. I'm happy to do this work if there's general interest in adding >> as3 support - let me know. > > -- > This message is automatically generated by JIRA. > - > You can reply to this email to add a comment to the issue online. > >
