Skip to content

Commit

Permalink
GOCBC-1068: Change preempted config update log level to warn
Browse files Browse the repository at this point in the history
Motivation
----------
When two configs are attempted to be applied at the same time we
reject one config and log an error. Whilst this is a case that
should not occur we should drop the log level to warn from error.
This happening can be indicative of something untoward happening
but in itself is not an error which causes the SDK to work incorrectly.

Changes
-------
Update preemptive config update log level to warn.

Change-Id: I3983c8c9a2cfcde1641d44569bf109e7f151327c
Reviewed-on: http://review.couchbase.org/c/gocbcore/+/147909
Tested-by: Charles Dixon <chvckd@gmail.com>
Reviewed-by: Brett Lawson <brett19@gmail.com>
  • Loading branch information
chvck committed Mar 8, 2021
1 parent 7e49c85 commit cd8d258
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion kvmux.go
Expand Up @@ -102,7 +102,7 @@ func (mux *kvMux) OnNewRouteConfig(cfg *routeConfig) {

// Attempt to atomically update the routing data
if !mux.updateState(oldMuxState, newMuxState) {
logErrorf("Someone preempted the config update, skipping update")
logWarnf("Someone preempted the config update, skipping update")
return
}

Expand Down

0 comments on commit cd8d258

Please sign in to comment.