[ 
https://issues.apache.org/jira/browse/THRIFT-1478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178069#comment-13178069
 ] 

Andrew Grumet commented on THRIFT-1478:
---------------------------------------

Yep!  I'm a bit of a n00b to this stuff too, so I've posted my process here to 
ensure correctness:

http://mail-archives.apache.org/mod_mbox/thrift-dev/201112.mbox/browser

Assuming that's ok, you'll need something like this for the server endpoint 
(note that the paths on your system will be different):

http://pastebin.com/LnAw902b

Also, I ran the tests, results are here: http://imm.io/dPTx

The i64 errors are probably due to me testing on a 32 bit machine.  Not sure 
yet how best to deal with that, but my thought is to return strings since 
that's what the PHP bc stuff expects.

Also, there are errors with the more complex structs.  I'm pretty sure you've 
got the same bug in pushContext that we ran into.  It's pretty subtle.  You 
actually want to push the current context, $this->context_, onto the stack and 
not $c.




                
> TJSONProtocol in PHP
> --------------------
>
>                 Key: THRIFT-1478
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1478
>             Project: Thrift
>          Issue Type: Improvement
>          Components: PHP - Library
>    Affects Versions: 0.8
>            Reporter: Greg Fodor
>            Priority: Minor
>         Attachments: THRIFT-1478.patch
>
>
> We needed to be able to serialize and deserialize javascript thrift calls 
> from within PHP, so I spent a few hours porting over the Java implementation 
> of TJSONProtocol to PHP. It has not been exhaustively tested but it seems to 
> be in reasonable working order. Please let me know what steps are necessary 
> to get this onto trunk. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to