imay commented on a change in pull request #1954: Limit the memory usage of
Loading process
URL: https://github.com/apache/incubator-doris/pull/1954#discussion_r333940204
##########
File path: be/src/runtime/tablets_channel.h
##########
@@ -73,10 +75,15 @@ class TabletsChannel {
const google::protobuf::RepeatedField<int64_t>& partition_ids,
google::protobuf::RepeatedPtrField<PTabletInfo>* tablet_vec);
- time_t last_updated_time() {
- return _last_updated_time;
- }
+ Status cancel();
+ // upper application may call this to try to reduce the mem usage of this
channel.
+ // eg. flush the largest memtable immediately.
+ // return Status::OK if mem is reduced.
+ Status reduce_mem_usage();
+
+ int64_t mem_consumption() { return _mem_tracker->consumption(); }
Review comment:
```suggestion
int64_t mem_consumption() const { return _mem_tracker->consumption(); }
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
