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

Posted to dev@drill.apache.org by ppadma <gi...@git.apache.org> on 2017/01/04 04:08:48 UTC

[GitHub] drill pull request #718: DRILL-5127: Revert the fix for DRILL-4831

GitHub user ppadma opened a pull request:

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

    DRILL-5127: Revert the fix for DRILL-4831

    Please see DRILL-5127 for details.

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

    $ git pull https://github.com/ppadma/drill DRILL-5127

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

    https://github.com/apache/drill/pull/718.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 #718
    
----
commit e24a269bcfeee3e9d078a91b56aa36fd321d46be
Author: Padma Penumarthy <pp...@yahoo.com>
Date:   2017-01-03T22:01:00Z

    DRILL-5127: Revert the fix for DRILL-4831

----


---
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 #718: DRILL-5127: Revert the fix for DRILL-4831

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

Github user asfgit closed the pull request at:

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


---
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 #718: DRILL-5127: Revert the fix for DRILL-4831

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

Github user amansinha100 commented on the issue:

    https://github.com/apache/drill/pull/718
  
    Please make sure to run DRILL-5082 test to confirm if the regression is fixed. 


---
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 #718: DRILL-5127: Revert the fix for DRILL-4831

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

Github user amansinha100 commented on the issue:

    https://github.com/apache/drill/pull/718
  
    I suppose we need more time to investigate why the combination of DRILL-4831 plus DRILL-5082 fixes causes couple of other issues to show up at high concurrency.   I am ok with reverting this for now. 
    +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.
---