You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by laurentgo <gi...@git.apache.org> on 2017/03/20 18:49:14 UTC

[GitHub] drill pull request #791: DRILL-5369: Add initializer for ServerMetaContext

GitHub user laurentgo opened a pull request:

    https://github.com/apache/drill/pull/791

    DRILL-5369: Add initializer for ServerMetaContext

    ServerMetaContext had no default constructor. The lack of it
    might cause m_done to be set to true, same for other variables.
    
    Add a default constructor to explicitly initialize its members.

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

    $ git pull https://github.com/laurentgo/drill laurent/DRILL-5369

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

    https://github.com/apache/drill/pull/791.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 #791
    
----
commit a4dc9e853e7a986f8fca8434f0e65b4d7a129918
Author: Laurent Goujon <la...@dremio.com>
Date:   2017-03-20T18:46:58Z

    DRILL-5369: Add initializer for ServerMetaContext
    
    ServerMetaContext had no default constructor. The lack of it
    might cause m_done to be set to true, same for other variables.
    
    Add a default constructor to explicitly initialize its members.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill pull request #791: DRILL-5369: Add initializer for ServerMetaContext

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/drill/pull/791


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill issue #791: DRILL-5369: Add initializer for ServerMetaContext

Posted by sudheeshkatkam <gi...@git.apache.org>.

Github user sudheeshkatkam commented on the issue:

    https://github.com/apache/drill/pull/791
  
    +1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill issue #791: DRILL-5369: Add initializer for ServerMetaContext

Posted by superbstreak <gi...@git.apache.org>.

Github user superbstreak commented on the issue:

    https://github.com/apache/drill/pull/791
  
    Thanks @laurentgo for the fix!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---