mymeiyi commented on code in PR #27726:
URL: https://github.com/apache/doris/pull/27726#discussion_r1434706359
##########
be/src/olap/wal_writer.h:
##########
@@ -34,34 +34,27 @@ extern const uint32_t k_wal_magic_length;
class WalWriter {
public:
- explicit WalWriter(const std::string& file_name,
- const std::shared_ptr<std::atomic_size_t>&
all_wal_disk_bytes,
- const std::shared_ptr<std::condition_variable>& cv);
+ explicit WalWriter(const std::string& file_name);
~WalWriter();
Status init();
Status finalize();
Status append_blocks(const PBlockArray& blocks);
- size_t disk_bytes() const { return
_disk_bytes.load(std::memory_order_relaxed); };
Status append_header(uint32_t version, std::string col_ids);
std::string file_name() { return _file_name; };
public:
static const int64_t LENGTH_SIZE = 8;
static const int64_t CHECKSUM_SIZE = 4;
- std::shared_ptr<std::condition_variable> cv;
static const int64_t VERSION_SIZE = 4;
private:
static constexpr size_t MAX_WAL_WRITE_WAIT_TIME = 1000;
std::string _file_name;
io::FileWriterPtr _file_writer;
- std::atomic_size_t _disk_bytes;
- std::shared_ptr<std::atomic_size_t> _all_wal_disk_bytes;
std::mutex _mutex;
Review Comment:
is this unsed?
--
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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
