Skip to content

Commit 9f99887

Browse files
Generate bucket_config_changed event log only when ...
there is a real change in bucket config. Change-Id: I016e1148cb70123c44f252aa45e2283c9253abb1 Reviewed-on: https://review.couchbase.org/c/ns_server/+/166031 Well-Formed: Build Bot <[email protected]> Tested-by: Build Bot <[email protected]> Tested-by: Hareen Kancharla <[email protected]> Reviewed-by: Steve Watanabe <[email protected]>
1 parent 138bd8e commit 9f99887

File tree

1 file changed

+16
-10
lines changed

1 file changed

+16
-10
lines changed

src/ns_bucket.erl

Lines changed: 16 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1001,16 +1001,22 @@ update_bucket_props(Type, StorageMode, BucketName, Props) ->
10011001
ok ->
10021002
{ok, NewBucketConfig} = ns_bucket:get_bucket(BucketName),
10031003
NewProps = extract_bucket_props(NewBucketConfig),
1004-
event_log:add_log(bucket_cfg_changed,
1005-
[{bucket, list_to_binary(BucketName)},
1006-
{bucket_uuid, uuid(BucketName, direct)},
1007-
{type, DisplayBucketType},
1008-
{old_settings,
1009-
{struct, build_bucket_props_json(
1010-
PrevProps)}},
1011-
{new_settings,
1012-
{struct, build_bucket_props_json(
1013-
NewProps)}}]),
1004+
if
1005+
PrevProps =/= NewProps ->
1006+
event_log:add_log(
1007+
bucket_cfg_changed,
1008+
[{bucket, list_to_binary(BucketName)},
1009+
{bucket_uuid, uuid(BucketName, direct)},
1010+
{type, DisplayBucketType},
1011+
{old_settings,
1012+
{struct, build_bucket_props_json(
1013+
PrevProps)}},
1014+
{new_settings,
1015+
{struct, build_bucket_props_json(
1016+
NewProps)}}]);
1017+
true ->
1018+
ok
1019+
end,
10141020
ok;
10151021
_ ->
10161022
RV

0 commit comments

Comments
 (0)