You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/05/10 07:00:22 UTC

[jira] [Commented] (TAJO-2145) Error codes based on errno.h need prefix

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

ASF GitHub Bot commented on TAJO-2145:
--------------------------------------

GitHub user eminency opened a pull request:

    https://github.com/apache/tajo/pull/1014

    TAJO-2145: Error codes based on errno.h need prefix

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/eminency/tajo TAJO-2145

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tajo/pull/1014.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 #1014
    
----
commit dd1e36aa86b2ef4655f0157b6dba04d330e7a586
Author: Jongyoung Park <em...@gmail.com>
Date:   2016-05-10T06:56:55Z

    Prefix 'CERR_' is added to some error codes

----


> Error codes based on errno.h need prefix
> ----------------------------------------
>
>                 Key: TAJO-2145
>                 URL: https://issues.apache.org/jira/browse/TAJO-2145
>             Project: Tajo
>          Issue Type: Task
>          Components: RPC
>    Affects Versions: 0.11.2
>            Reporter: Jongyoung Park
>            Assignee: Jongyoung Park
>             Fix For: 0.11.3
>
>
> Some error codes in errors.proto which are based on C standard library errno.h can be conflicted if it is used in C/C++ client.
> So, it needs to be changed by such as adding prefix.



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