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

[refactor] Change ALL OLAPStatus to Status (#8855)

Browse Source 
Currently, there are 2 status code in BE, one is common/Status.h,
and the other is olap/olap_define.h called OLAPStatus.
OLAPStatus is just an enum type, it is very simple and could not save many informations,
I will unify these code to common/Status.
This commit is contained in:
yiguolei
2022-04-14 11:43:49 +08:00
committed by GitHub
parent 8765881d8b
commit e5e0dc421d
212 changed files with 4754 additions and 5148 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
be/src/exec/olap_scan_node.cpp
View File

@ -703,11 +703,11 @@ static Status get_hints(TabletSharedPtr table, const TPaloScanRange& scan_range,
}
SCOPED_TIMER(show_hints_timer);
OLAPStatus res = OLAP_SUCCESS;
Status res = Status::OK();
std::vector<OlapTuple> range;
res = table->split_range(key_range->begin_scan_range, key_range->end_scan_range,
block_row_count, &range);
if (res != OLAP_SUCCESS) {
if (!res.ok()) {
return Status::InternalError("fail to show hints");
}
ranges.emplace_back(std::move(range));
@ -717,7 +717,7 @@ static Status get_hints(TabletSharedPtr table, const TPaloScanRange& scan_range,
if (!have_valid_range) {
std::vector<OlapTuple> range;
auto res = table->split_range({}, {}, block_row_count, &range);
if (res != OLAP_SUCCESS) {
if (!res.ok()) {
return Status::InternalError("fail to show hints");
}
ranges.emplace_back(std::move(range));