You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2015/03/19 03:04:38 UTC

[jira] [Commented] (THRIFT-2928) Rename the erlang test_server module

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

ASF GitHub Bot commented on THRIFT-2928:
----------------------------------------

Github user jfarrell commented on the pull request:

    https://github.com/apache/thrift/pull/346#issuecomment-83265022
  
    THRIFT-2928 closed, thanks for the patch


> Rename the erlang test_server module
> ------------------------------------
>
>                 Key: THRIFT-2928
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2928
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Erlang - Library
>            Reporter: Bhasker Kode
>            Assignee: Roger Meier
>            Priority: Minor
>
> erlang/otp began shipping with an app called *test_server* since November 2009 with R13B03. The current version of erlang/otp is approaching R18.
> Thrift's test also comes with test_server.erl at {{lib/erl/test/test_server.erl}} as well as a test_client.erl
> after compiling Thrift's erlang modules, I was expecting to find {{test_server:go/0}} but instead found functions from {{https://github.com/erlang/otp/blob/master/lib/test_server/src/test_server.erl}}
> To avoid such conflicts while developing and in erlang deployments/releases, {{test_server.erl}} can be renamed as {{test_thrift_server.erl}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)