[
https://issues.apache.org/jira/browse/THRIFT-4187?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16010602#comment-16010602
]
ASF GitHub Bot commented on THRIFT-4187:
----------------------------------------
Github user markerickson-wf commented on the issue:
https://github.com/apache/thrift/pull/1269
@allengeorge I think this approach looks pretty good at my first pass. I
left a couple of comments, and we'll probably want unit tests around the state
transitions to ensure that all paths are covered (setting / unsetting
completers, resetting read counters, etc). I'll also ask couple of other devs
I know to review (they have done some recent work using framed transports in
Dart). Thanks for submitting the PR!
> Dart -> Rust Framed cross tests fail
> ------------------------------------
>
> Key: THRIFT-4187
> URL: https://issues.apache.org/jira/browse/THRIFT-4187
> Project: Thrift
> Issue Type: Bug
> Components: Rust - Library
> Reporter: Allen George
> Assignee: Allen George
> Priority: Minor
>
> For some reason the Dart (client) -> Rust (server) framed-transport cross
> tests fail. Initial investigation shows that *somehow* the dart client think
> the socket was closed, which means it doesn't read the message from the
> underlying transport.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
