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

BLZG-9177 Use slf4j for logging errors in BigdataRDFServlet #147

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

igor-kim
Copy link
Contributor

@igor-kim igor-kim commented Dec 4, 2019

  • migrate logger to slf4j (this should be transparent for all use cases,
    blazegraph already packages slf4j-log4j12)
  • replace string concatenation in log message with placeholders (this
    will allow better management of logs, since message format will
    be available in log processin)

Change-Id: Ic4c2ba04ae0be35396929e468b7c54cb716af53d

* migrate logger to slf4j (this should be transparent for all use cases,
blazegraph already packages slf4j-log4j12)
* replace string concatenation in log message with placeholders (this
  will allow better management of logs, since message format will
  be available in log processin)

Change-Id: Ic4c2ba04ae0be35396929e468b7c54cb716af53d
@igor-kim igor-kim closed this Dec 4, 2019
@igor-kim igor-kim reopened this Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants