You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Bryan Duxbury (JIRA)" <ji...@apache.org> on 2010/07/13 18:10:49 UTC

[jira] Created: (THRIFT-818) Async client doesn't send method args

Async client doesn't send method args
-------------------------------------

                 Key: THRIFT-818
                 URL: https://issues.apache.org/jira/browse/THRIFT-818
             Project: Thrift
          Issue Type: Bug
          Components: Library (Java)
            Reporter: Bryan Duxbury
            Assignee: Bryan Duxbury
             Fix For: 0.4


It turns out there was a stupid mistake in the unit tests for TAsyncClientManager that was preventing us from noticing that method args weren't getting sent to the server. The actual problem turned out to be an order of events issue that's easily fixed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (THRIFT-818) Async client doesn't send method args

Posted by "Bryan Duxbury (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/THRIFT-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bryan Duxbury closed THRIFT-818.
--------------------------------

    Resolution: Fixed

I just committed this.

> Async client doesn't send method args
> -------------------------------------
>
>                 Key: THRIFT-818
>                 URL: https://issues.apache.org/jira/browse/THRIFT-818
>             Project: Thrift
>          Issue Type: Bug
>          Components: Library (Java)
>            Reporter: Bryan Duxbury
>            Assignee: Bryan Duxbury
>             Fix For: 0.4
>
>         Attachments: thrift-818.patch
>
>
> It turns out there was a stupid mistake in the unit tests for TAsyncClientManager that was preventing us from noticing that method args weren't getting sent to the server. The actual problem turned out to be an order of events issue that's easily fixed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (THRIFT-818) Async client doesn't send method args

Posted by "Bryan Duxbury (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/THRIFT-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bryan Duxbury updated THRIFT-818:
---------------------------------

    Attachment: thrift-818.patch

This patch fixes both the test and the underlying code. 

> Async client doesn't send method args
> -------------------------------------
>
>                 Key: THRIFT-818
>                 URL: https://issues.apache.org/jira/browse/THRIFT-818
>             Project: Thrift
>          Issue Type: Bug
>          Components: Library (Java)
>            Reporter: Bryan Duxbury
>            Assignee: Bryan Duxbury
>             Fix For: 0.4
>
>         Attachments: thrift-818.patch
>
>
> It turns out there was a stupid mistake in the unit tests for TAsyncClientManager that was preventing us from noticing that method args weren't getting sent to the server. The actual problem turned out to be an order of events issue that's easily fixed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.