Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audit logger thread is blocked under high load #399

Closed
yuriyz opened this issue Dec 19, 2016 · 6 comments
Closed

Audit logger thread is blocked under high load #399

yuriyz opened this issue Dec 19, 2016 · 6 comments
Assignees
Labels
bug bug in code
Milestone

Comments

@yuriyz
Copy link
Contributor

yuriyz commented Dec 19, 2016

With 200 concurrent threads and enabled audit logger eventually thread is blocked. Audit event must be logged in asynchronous way without blocking current thread.

@yuriyz yuriyz added the bug bug in code label Dec 19, 2016
@yuriyz yuriyz added this to the CE 3.0.0 milestone Dec 19, 2016
@yuriyz
Copy link
Contributor Author

yuriyz commented Dec 19, 2016

audit_logger_1

@yuriyz
Copy link
Contributor Author

yuriyz commented Dec 19, 2016

@yurem
Copy link
Contributor

yurem commented Dec 20, 2016

We inject configuration into bean now

@yurem yurem closed this as completed Dec 20, 2016
@yuriyz
Copy link
Contributor Author

yuriyz commented Dec 20, 2016

Next bottleneck would be sendMessage which is implemented in synchronous way while we need to work it in asynchrounous way. E.g. via Blocking queue.

@yuriyz yuriyz reopened this Dec 20, 2016
@yurem
Copy link
Contributor

yurem commented Dec 20, 2016

sendMessage is not synchronous

	@Asynchronous
	public void sendMessage(OAuth2AuditLog oAuth2AuditLog) {

@yuriyz
Copy link
Contributor Author

yuriyz commented Dec 20, 2016

ah, right. Closing for now, we will check it with profiler.

@yuriyz yuriyz closed this as completed Dec 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bug in code
Projects
None yet
Development

No branches or pull requests

2 participants