morningman commented on a change in pull request #3632:
URL: https://github.com/apache/incubator-doris/pull/3632#discussion_r427033285
##########
File path: be/src/olap/data_dir.cpp
##########
@@ -694,19 +694,26 @@ OLAPStatus DataDir::load() {
// create tablet from tablet meta and add it to tablet mgr
LOG(INFO) << "begin loading tablet from meta";
std::set<int64_t> tablet_ids;
- auto load_tablet_func = [this, &tablet_ids](int64_t tablet_id, int32_t
schema_hash,
+ std::set<int64_t> failed_tablet_ids;
+ auto load_tablet_func = [this, &tablet_ids, &failed_tablet_ids](int64_t
tablet_id, int32_t schema_hash,
const std::string& value) ->
bool {
OLAPStatus status = _tablet_manager->load_tablet_from_meta(this,
tablet_id, schema_hash,
value,
false, false);
if (status != OLAP_SUCCESS) {
LOG(WARNING) << "load tablet from header failed. status:" << status
<< ", tablet=" << tablet_id << "." << schema_hash;
+ failed_tablet_ids.insert(tablet_id);
} else {
tablet_ids.insert(tablet_id);
}
return true;
};
OLAPStatus load_tablet_status = TabletMetaManager::traverse_headers(_meta,
load_tablet_func);
+ if (failed_tablet_ids.size() != 0 && !config::ignore_load_tablet_failure) {
+ std::stringstream ss;
+ std::copy(failed_tablet_ids.begin(), failed_tablet_ids.end(),
std::ostream_iterator<int64_t>(ss, ","));
Review comment:
There may be thousands of tablets. This time the log is very large.
In addition, the problematic tablet has actually been printed in the log, so
there is no need to print the tablet id. Just print size is enough
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
