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;
}