database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[Enhencement](Broker reader)Use smart Pointer instead of native Pointers in broker reader

Browse Source
This commit is contained in:
Tiewei Fang
2023-02-13 21:01:53 +08:00
committed by GitHub
parent 15d9dd114b
commit c620e06f6a
2 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
be/src/io/broker_reader.cpp
View File

@ -77,8 +77,7 @@ Status BrokerReader::open() {
request.__set_clientId(client_id(_env, broker_addr));
request.__set_properties(_properties);
TBrokerOpenReaderResponse* response = new TBrokerOpenReaderResponse();
Defer del_reponse {[&] { delete response; }};
std::unique_ptr<TBrokerOpenReaderResponse> response(new TBrokerOpenReaderResponse());
try {
Status status;
BrokerServiceConnection client(client_cache(_env), broker_addr,

3
be/src/io/fs/broker_file_system.cpp
View File

@ -94,8 +94,7 @@ Status BrokerFileSystem::open_file(const Path& path, FileReaderSPtr* reader,
request.__set_clientId(client_id(_broker_addr));
request.__set_properties(_broker_prop);
TBrokerOpenReaderResponse* response = new TBrokerOpenReaderResponse();
Defer del_reponse {[&] { delete response; }};
std::unique_ptr<TBrokerOpenReaderResponse> response(new TBrokerOpenReaderResponse());
try {
Status status;
try {