[ https://issues.apache.org/jira/browse/THRIFT-3631?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15146367#comment-15146367 ]
ASF GitHub Bot commented on THRIFT-3631: ---------------------------------------- GitHub user mygityf reopened a pull request: https://github.com/apache/thrift/pull/863 THRIFT-3631 Lua-This is JSON protocol implement for thrift-lua library. TJsonProtocol.lua is new source of lua for thrift JSON protocol. Test result is expected of test_basic_server.lua, test_basic_client.lua. Test cases: client <--------------> server json_protocol_client.lua json_server.go (golang thrift server) json_client.go (golang thrift client) json_protocol_server.lua json_protocol_client.lua json_protocol_server.lua test_basic_server.lua test_basic_client.lua You can merge this pull request into a Git repository by running: $ git pull https://github.com/gityf/thrift master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/thrift/pull/863.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #863 ---- commit 0d0cf5aa589389d40a2bd71a9e09e790634aff36 Author: unknown <wangya...@xz-20141016zrqg.didichuxing.com> Date: 2016-01-28T11:08:05Z To check element of self whether nil or not,avoid the value(false) of BOOL is lost. eg. Before: if self.argBool then oprot:writeFieldBegin('argBool', TType.BOOL, 13) oprot:writeBool(self.argBool) oprot:writeFieldEnd() end After: if self.argBool ~= nil then oprot:writeFieldBegin('argBool', TType.BOOL, 13) oprot:writeBool(self.argBool) oprot:writeFieldEnd() end Why: the value(false) for paramater self.argBool will be lost when condition is 'if self.argBool then' as before. commit 8e61f5ae5cd82ede21d2fad0c67e9328a57a9bc5 Author: unknown <wangya...@xz-20141016zrqg.didichuxing.com> Date: 2016-01-28T11:29:54Z This is compact protocol lua implement for thrift. Here are some description for each source code file. luabpack.c To add bits operation function for thrift compact protocol. new functions: l_i64ToZigza,l_i32ToZigzag,l_zigzagToI32,l_zigzagToI64,l_toVarint32,l_toVarint64,l_packMesgType. TCompactProtocol.lua new source of lua for thrift compact protocol. compact_protocol_client.lua test client source code for thrift compact protocol. compact_protocol_server.lua test server source code for thrift compact protocol. rpc.thrift test thrift IDL file for test case(compact_protocol_client.lua,compact_protocol_server.lua). pcap offset hex of rpc test service.txt This is dump file create by tcpdump for test case. Test cases: client <--------------> server compact_protocol_client.lua compact_server.go (golang thrift server) compact_client.go (golang thrift client) compact_protocol_server.lua compact_protocol_client.lua compact_protocol_server.lua Test console log: log for compact_protocol_client.lua: 1 return 1 by FunCall. 2 return 2 by FunCall. log for compact_protocol_server.lua: 53 54 12 43 11.22 str value 53 54 12 34 11.22 login name namess pass vpass 10 val10 20 val20 ele1 ele3 ele2 l1. l2. commit df693bf78aaf9fd37eff5d972ed6225cd5e88e38 Author: WangYaofu <voip...@sina.cn> Date: 2016-02-04T07:29:36Z THRIFT-3585 Lua: compact protocol implement of lua for thrift. To fix bug: Examples of problematic valus were negative i8 values (-128 ~ -1), some i32 values (0x40000000, -1 ...), i64 values. commit 982c519b855a6fcdb1b8b49951cd6f8d39c8c8bb Author: Wang Yaofu <voip...@sina.cn> Date: 2016-02-04T07:40:37Z THRIFT-3585 Lua: compact protocol implement of lua for thrift. #ifdef against Lua version macro added. lua_objlen for version macro _LUA51_ for lua5.1. lua_rawlen for lua5.2+. commit 880e559e185d24005893d412f65e0199656e89a2 Author: WangYaofu <voip...@sina.cn> Date: 2016-02-14T03:26:36Z THRIFT-3631 Lua-This is JSON protocol implement for thrift-lua library. TJsonProtocol.lua is new source of lua for thrift JSON protocol. ---- > JSON protocol implementation for Lua > ------------------------------------ > > Key: THRIFT-3631 > URL: https://issues.apache.org/jira/browse/THRIFT-3631 > Project: Thrift > Issue Type: Improvement > Components: Lua - Library > Affects Versions: 0.9.3 > Environment: Linux Mac Windows > Reporter: WangYaofu > Assignee: WangYaofu > Fix For: 0.9.4 > > > This is JSON protocol implement for thrift-lua library. > TJsonProtocol.lua is new source of lua for thrift JSON protocol. -- This message was sent by Atlassian JIRA (v6.3.4#6332)