Issue Details (XML | Word | Printable)

Key: CONF-10621
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Matt Ryall [Atlassian]
Reporter: Matt Ryall [Atlassian]
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Confluence

Improve logging in DefaultSpacePermissionManager

Created: 31/Jan/08 05:32 PM   Updated: 08/Feb/08 12:00 AM
Component/s: Engine Room / Architecture, Logging & Auditing, Support
Affects Version/s: 2.7.1
Fix Version/s: 2.7.2

Time Tracking:
Not Specified

File Attachments: 1. File DefaultSpacePermissionManager.class (14 kB)


Participants: Charles Miller [old account, do not assign issues] and Matt Ryall [Atlassian]
Since last comment: 42 weeks, 5 days ago
Resolution Date: 08/Feb/08 12:00 AM
Labels:


 Description  « Hide
There's not much profiling or debug logging inside the DefaultSpacePermissionManager, yet it is often slow to process requests.

We should add some better logging here (and probably profiling too) for support purposes.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Matt Ryall [Atlassian] added a comment - 04/Feb/08 11:32 PM
Patched DefaultSpacePermissionManager from stable branch.

Should be suitable for applying to Confluence 2.7 and 2.7.1.


Charles Miller [old account, do not assign issues] added a comment - 05/Feb/08 11:56 PM
Assigning to Matt, since he told me he had code ready to check in for this...

Charles Miller [old account, do not assign issues] added a comment - 05/Feb/08 11:57 PM
Oops. That'll teach me to read the comments before hitting the assign button.

What's the next step? Do we want this for the stable branch? Who owns it?


Matt Ryall [Atlassian] added a comment - 06/Feb/08 12:33 AM
I still have to merge to trunk. Will do so as soon as my trunk build is working. :/