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

Posted to dev@drill.apache.org by StevenMPhillips <gi...@git.apache.org> on 2015/12/21 22:04:54 UTC

[GitHub] drill pull request: DRILL-4215: Transfer ownership in TransferPair

GitHub user StevenMPhillips opened a pull request:

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

    DRILL-4215: Transfer ownership in TransferPair

    Only commit 79bbfcb is part of this commit. But since the other commits haven't been merged to master, they are also showing up in this PR.

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

    $ git pull https://github.com/StevenMPhillips/drill drill-4215

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

    https://github.com/apache/drill/pull/308.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 #308
    
----
commit ea7bbc58f3e4bed5dc8093381d6c7dc4911f520e
Author: Chris Westin <cw...@yahoo.com>
Date:   2015-11-11T22:57:16Z

    DRILL-4144: Clean up close behaviors.

commit da4f9821644520bf2c4eb698770ed87713b1a039
Author: Chris Westin <cw...@yahoo.com>
Date:   2015-11-11T22:57:47Z

    DRILL-4144: Add new allocator

commit 728687f8d7c28d398ac993705dc0e4fb189b01d4
Author: Jacques Nadeau <ja...@apache.org>
Date:   2015-11-16T01:26:02Z

    DRILL-4144: Allocator Improvements
    
    - make Allocator mostly lockless
    - change BaseAllocator maps to direct references
    - add documentation around memory management model
    - move transfer and ownership methods to DrillBuf
    - Improve debug messaging.
    - Fix/revert sort changes
    - Remove unused fragment limit flag
    - Add time to HistoricalLog events
    - Remove reservation amount from RootAllocator constructor (since not allowed)
    - Fix concurrency issue where allocator is closing at same moment as incoming batch transfer, causing leaked memory and/or query failure.
    - Add new AutoCloseables.close(Iterable<AutoCloseable>)
    - Remove extraneous DataResponseHandler and Impl (and update TestBitRpc to use smarter mock of FragmentManager)
    - Remove the concept of poison pill record batches, using instead FragmentContext.isOverMemoryLimit()
    - Update incoming data batches so that they are transferred under protection of a close lock
    - Improve field names in IncomingBuffers and move synchronization to collectors as opposed to IncomingBuffers (also change decrementing to decrementToZero rather than two part check).

commit 79bbfcb018429ad3a02daf2ac4dea119126041e1
Author: Steven Phillips <sm...@apache.org>
Date:   2015-12-01T08:34:41Z

    DRILL-4215: Transfer buffer ownership in TransferPair

----


---
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: DRILL-4215: Transfer ownership in TransferPair

Posted by jacques-n <gi...@git.apache.org>.

Github user jacques-n commented on the pull request:

    https://github.com/apache/drill/pull/308#issuecomment-170455457
  
    @StevenMPhillips, can you close?


---
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.
---