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