Skip to content

Commit

Permalink
Sandesh client SM task needs to be run exclusive to sandesh client
Browse files Browse the repository at this point in the history
reader and writer tasks. The sandesh client reader task was named
io::ReaderTask which is the same as the collector reader task and
hence in high scale the sandesh client SM task would be in deferq
and not run since a lot of io::ReaderTask would always be running.
This causes collector statistics to not be sent. Fix is to rename
the sandesh client reader task to sandesh::SandeshClientReader.
Partial-Bug: #1469919

Change-Id: I728d7054d3d1d14c3fca3dc6e0d961c3090a616a
  • Loading branch information
Megh Bhatt committed Jul 15, 2015
1 parent 557f16a commit 0808b1d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion library/cpp/sandesh_client.cc
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ using boost::system::error_code;

const std::string SandeshClient::kSMTask = "sandesh::SandeshClientSM";
const std::string SandeshClient::kSessionWriterTask = "sandesh::SandeshClientSession";
const std::string SandeshClient::kSessionReaderTask = "io::ReaderTask";
const std::string SandeshClient::kSessionReaderTask = "sandesh::SandeshClientReader";
bool SandeshClient::task_policy_set_ = false;
const std::vector<Sandesh::QueueWaterMarkInfo>
SandeshClient::kSessionWaterMarkInfo = boost::assign::tuple_list_of
Expand Down

0 comments on commit 0808b1d

Please sign in to comment.