Skip to content

Commit

Permalink
Reduce default logging from reactor.
Browse files Browse the repository at this point in the history
Summary: Noticed too much of this logging in CE tests of itest.

Test Plan: Jenkins: compile-only

Reviewers: mikhail, bogdan

Reviewed By: bogdan

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D4661
  • Loading branch information
bbaddepudi committed Apr 24, 2018
1 parent be18db9 commit aad076c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/yb/common/flags.cc
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ void InitCommonFlags() {
} else if (std::thread::hardware_concurrency() <= 2) {
value = 4;
}
LOG(INFO) << "Auto setting FLAGS_yb_num_shards_per_tserver to " << value;
VLOG(1) << "Auto setting FLAGS_yb_num_shards_per_tserver to " << value;
FLAGS_yb_num_shards_per_tserver = value;
}
}
Expand Down
6 changes: 3 additions & 3 deletions src/yb/rpc/reactor.cc
Original file line number Diff line number Diff line change
Expand Up @@ -296,18 +296,18 @@ void Reactor::CheckReadyToStop() {
if (waiting_conns_.empty()) {
VLOG(4) << "Reactor ready to stop, breaking loop: " << this;

LOG(INFO) << "Marking reactor as closed: " << thread_.get()->ToString();
VLOG(2) << "Marking reactor as closed: " << thread_.get()->ToString();
ReactorTasks final_tasks;
{
std::lock_guard<simple_spinlock> lock(pending_tasks_mtx_);
state_.store(ReactorState::kClosed, std::memory_order_release);
final_tasks.swap(pending_tasks_);
}
LOG(INFO) << "Running final pending task aborts: " << thread_.get()->ToString();;
VLOG(2) << "Running final pending task aborts: " << thread_.get()->ToString();;
for (auto task : final_tasks) {
task->Abort(ServiceUnavailableError());
}
LOG(INFO) << "Breaking reactor loop: " << thread_.get()->ToString();;
VLOG(2) << "Breaking reactor loop: " << thread_.get()->ToString();;
loop_.break_loop(); // break the epoll loop and terminate the thread
}
}
Expand Down

0 comments on commit aad076c

Please sign in to comment.