You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by pc...@apache.org on 2019/01/19 21:35:29 UTC

[arrow] branch master updated: ARROW-4295: [C++] [Plasma] Fix incorrect log message

This is an automated email from the ASF dual-hosted git repository.

pcmoritz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new cc1ce61  ARROW-4295: [C++] [Plasma] Fix incorrect log message
cc1ce61 is described below

commit cc1ce6194b905768b1a6d9f0e209270f62dc558a
Author: Anurag Khandelwal <an...@berkeley.edu>
AuthorDate: Sat Jan 19 13:35:10 2019 -0800

    ARROW-4295: [C++] [Plasma] Fix incorrect log message
    
    https://issues.apache.org/jira/browse/ARROW-4295
    
    Fixes incorrect message printed when evicting objects in plasma.
    
    Author: Anurag Khandelwal <an...@berkeley.edu>
    
    Closes #3433 from anuragkh/ARROW-4295 and squashes the following commits:
    
    8c52725e8 <Anurag Khandelwal> Fix incorrect log message
---
 cpp/src/plasma/eviction_policy.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/plasma/eviction_policy.cc b/cpp/src/plasma/eviction_policy.cc
index e5beb5a..4fb0cce 100644
--- a/cpp/src/plasma/eviction_policy.cc
+++ b/cpp/src/plasma/eviction_policy.cc
@@ -85,7 +85,7 @@ bool EvictionPolicy::RequireSpace(int64_t size, std::vector<ObjectID>* objects_t
   ARROW_LOG(INFO) << "There is not enough space to create this object, so evicting "
                   << objects_to_evict->size() << " objects to free up "
                   << num_bytes_evicted << " bytes. The number of bytes in use (before "
-                  << "this eviction) is " << memory_used_ << ".";
+                  << "this eviction) is " << (memory_used_ + num_bytes_evicted) << ".";
   return num_bytes_evicted >= required_space && num_bytes_evicted > 0;
 }