Skip to content

Commit

Permalink
Use term “application subscriptions” in log
Browse files Browse the repository at this point in the history
This patch completes the work of replacing the older term “auto subscription”
with the newer term “application subscription” by changing log lines that the
broker outputs.  This may break some external applications, so it is included
as its own patch.

Signed-off-by: Patrick M. Niedzielski <[email protected]>
  • Loading branch information
pniedzielski committed Dec 12, 2024
1 parent 36a1a57 commit 253ffde
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions src/groups/mqb/mqbblp/mqbblp_rootqueueengine.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -341,7 +341,7 @@ int RootQueueEngine::configure(bsl::ostream& errorDescription,
BALL_LOG_ERROR
<< "#QUEUE_CONFIGURE_FAILURE Queue '"
<< d_queueState_p->queue()->description()
<< "' failed to compile auto subscription: '"
<< "' failed to compile application subscription: '"
<< subscriptions[i].expression().text()
<< "' for the '" << itApp->first << "' app, rc: " << rc
<< ", reason: '"
Expand All @@ -352,7 +352,7 @@ int RootQueueEngine::configure(bsl::ostream& errorDescription,
else {
BALL_LOG_WARN << "Queue \""
<< d_queueState_p->queue()->description()
<< "' ignores auto subscription: '"
<< "' ignores application subscription: '"
<< subscriptions[i].appId() << "'";
}
}
Expand All @@ -370,7 +370,8 @@ int RootQueueEngine::configure(bsl::ostream& errorDescription,
if (subscriptions.size() > 1) {
BALL_LOG_ERROR << "#QUEUE_CONFIGURE_FAILURE Queue '"
<< d_queueState_p->queue()->description()
<< "' Cannot have more than 1 auto subscription";
<< "' Cannot have more than 1 application "
<< "subscription";

return rc_APP_SUBSCRIPTIONS_ERROR; // RETURN
}
Expand All @@ -393,7 +394,7 @@ int RootQueueEngine::configure(bsl::ostream& errorDescription,

BALL_LOG_ERROR << "#QUEUE_CONFIGURE_FAILURE Queue '"
<< d_queueState_p->queue()->description()
<< "' Failed to compile auto subscription: '"
<< "' Failed to compile application subscription: '"
<< subscriptions[0].expression().text()
<< "', rc: " << rc << ", reason: '"
<< bmqeval::ErrorType::toString(errorType) << "'";
Expand Down

0 comments on commit 253ffde

Please sign in to comment.