xinyiZzz commented on a change in pull request #8476:
URL: https://github.com/apache/incubator-doris/pull/8476#discussion_r829029793
##########
File path: be/src/olap/snapshot_manager.h
##########
@@ -99,6 +102,9 @@ class SnapshotManager {
// snapshot
Mutex _snapshot_mutex;
uint64_t _snapshot_base_id;
+
+ // TODO(zxy) used after
+ std::shared_ptr<MemTracker> _mem_tracker = nullptr;
Review comment:
In the next pr, this tracker will be switched to the tls mem tracker in
the public func of `SnapshotManager`.
In this pr, I created all the trackers that will be used in the future, and
built a complete mem tracker tree. (Perhaps it would be better to do this with
a pr alone... Do you think it needs to be deleted first? = =||| )
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
