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 2018/03/03 16:57:57 UTC

[GitHub] drill issue #1125: DRILL-6126: Allocate memory for value vectors upfront in ...

Github user ppadma commented on the issue:

    https://github.com/apache/drill/pull/1125
  
    @paul-rogers Updated the PR with latest changes.  I have decided not to use vector initializer for allocation as it is subject to alias issues like you mentioned. Instead, I added allocate vector method to columnSize in batch sizer, which will use internal sizing information it has to allocate memory (including all it's children) for a particular record count. 
    Refactored the batch sizer code, added unit tests for verifying sizing and allocation for different vector types. 
    Please take a look when you get a chance and let me know what you think.



---