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 2018/08/23 17:44:40 UTC

[arrow] branch master updated: ARROW-3105: [Plasma] Improve flushing error 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 fda4b3d  ARROW-3105: [Plasma] Improve flushing error message
fda4b3d is described below

commit fda4b3dcfc773612b12973df5053193f236fc696
Author: Robert Nishihara <ro...@gmail.com>
AuthorDate: Thu Aug 23 10:44:29 2018 -0700

    ARROW-3105: [Plasma] Improve flushing error message
    
    Author: Robert Nishihara <ro...@gmail.com>
    Author: Philipp Moritz <pc...@gmail.com>
    
    Closes #2458 from pcmoritz/plasma-flushing-message and squashes the following commits:
    
    f2d08296 <Robert Nishihara> Update eviction_policy.cc
    b3e19883 <Philipp Moritz> improve flushing error message
---
 cpp/src/plasma/eviction_policy.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/cpp/src/plasma/eviction_policy.cc b/cpp/src/plasma/eviction_policy.cc
index cace588..e5beb5a 100644
--- a/cpp/src/plasma/eviction_policy.cc
+++ b/cpp/src/plasma/eviction_policy.cc
@@ -84,7 +84,8 @@ bool EvictionPolicy::RequireSpace(int64_t size, std::vector<ObjectID>* objects_t
   int64_t num_bytes_evicted = ChooseObjectsToEvict(space_to_free, objects_to_evict);
   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.";
+                  << num_bytes_evicted << " bytes. The number of bytes in use (before "
+                  << "this eviction) is " << memory_used_ << ".";
   return num_bytes_evicted >= required_space && num_bytes_evicted > 0;
 }