You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Henrique Mendonça (JIRA)" <ji...@apache.org> on 2014/02/23 20:01:22 UTC
[jira] [Closed] (THRIFT-2350) Add async calls to normal JavaScript
[ https://issues.apache.org/jira/browse/THRIFT-2350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Henrique Mendonça closed THRIFT-2350.
-------------------------------------
> Add async calls to normal JavaScript
> ------------------------------------
>
> Key: THRIFT-2350
> URL: https://issues.apache.org/jira/browse/THRIFT-2350
> Project: Thrift
> Issue Type: Improvement
> Components: JavaScript - Compiler, JavaScript - Library
> Affects Versions: 0.9.2
> Environment: All
> Reporter: Randy Abernethy
> Assignee: Henrique Mendonça
> Priority: Minor
> Fix For: 0.9.2
>
> Attachments: 0001-add-async-support-to-default-JavaScript-client.patch, 0002-updated-grunfile-and-tests-for-async.patch, 0003-reorg-of-js-tests-removed-js-tests-from-node-dir.patch
>
>
> Currently async call are only available with -gen js if you unwire the client send/recv methods (pretty messy and unintuitive) or if you use jQuery (-gen js:jquery). This patch makes it easy to use async callbacks with any Thrift call by simply appending the desired callback to the args). e.g.
> client.myFunc(arg1, arg2, function (result) {
> //do callback stuff with result
> //result will be the normal return value or the exception
> });
> This patch preserves the existing sync call style (just leave off the callback).
> Compiler js generator changes
> =============================
> - Combining node and jquery switches now results in a compile error rather than silently producing corrupt code in ./gen-nodejs
> - Updated comments
> - Added ability to make async calls without jQuery
> Node test client.js
> ===================
> - Corrected comments and spelling
> JS thrift.js
> ====================
> - Added support for async calls w/o jQuery
> Node/JS test.js
> ====================
> - Updated comments to clarify jQuery dependency (-gen js:jquery)
> Node/JS test-nojq.js test-nojq.html
> ===================================
> - Full test suite for normal JS build (-gen js)
> Node http test server and handler
> =================================
> linted
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)