Skip to content

[13] Log lock state on conflict - #9389

Merged
MorrisJobke merged 2 commits into
stable13from
log-lock-state-on-conflict-13
May 7, 2018
Merged

[13] Log lock state on conflict#9389
MorrisJobke merged 2 commits into
stable13from
log-lock-state-on-conflict-13

Conversation

@icewind1991

@icewind1991icewind1991 commented May 4, 2018

Copy link
Copy Markdown
Member

Backport of #9388

@icewind1991
Log more info about locking conflicts for memcache locking backends
a070c11
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991
always log locking errors
bc1a17a
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991icewind1991 added the 3. to review Waiting for reviews label May 4, 2018
@icewind1991icewind1991 added this to the Nextcloud 13.0.3 milestone May 4, 2018
MorrisJobke
MorrisJobke approved these changes May 4, 2018
View reviewed changes

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@MorrisJobke
MorrisJobke merged commit 7bf713e into stable13May 7, 2018
@MorrisJobke
MorrisJobke deleted the log-lock-state-on-conflict-13 branch May 7, 2018 14:29
This was referenced May 31, 2018
Merged
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessennickvergessennickvergessen approved these changes
@MorrisJobkeMorrisJobkeMorrisJobke approved these changes
@rullzerrullzerAwaiting requested review from rullzer

Assignees

No one assigned

Labels

3. to reviewWaiting for reviews

Projects

None yet

Milestone

Nextcloud 13.0.3

Development

Successfully merging this pull request may close these issues.

3 participants

@icewind1991@nickvergessen@MorrisJobke