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

ASF GitHub Bot commented on ORC-21:
-----------------------------------

Github user omalley commented on a diff in the pull request:

    https://github.com/apache/orc/pull/12#discussion_r44307164
  
    --- Diff: c++/src/Vector.cc ---
    @@ -196,6 +246,17 @@ namespace orc {
         }
       }
     
    +  uint64_t MapVectorBatch::getMemoryUsage() {
    +    return ColumnVectorBatch::getMemoryUsage()
    +           + static_cast<uint64_t>(offsets.capacity() * sizeof(int64_t))
    +           + keys->getMemoryUsage();
    --- End diff --
    
    You have a spurious ';' here that is making your results and tests wrong.


> Add functionality to estimate memory footprint
> ----------------------------------------------
>
>                 Key: ORC-21
>                 URL: https://issues.apache.org/jira/browse/ORC-21
>             Project: Orc
>          Issue Type: Task
>            Reporter: Aliaksei Sandryhaila
>            Assignee: Aliaksei Sandryhaila
>
> ORC library allocates multiple large buffers to read and materialize ORC 
> files. For stability of applications that use the library, it may be 
> desirable to have an estimate (preferably, a tight upper bound) of a memory 
> footprint.



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

Reply via email to