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

Add log4j xml config #444

Merged
merged 2 commits into from
Dec 15, 2021
Merged

Add log4j xml config #444

merged 2 commits into from
Dec 15, 2021

Conversation

balhoff
Copy link
Member

@balhoff balhoff commented Dec 14, 2021

I realized that log4j 2.x requires a different config file format. However part of Blazegraph seems to independently load the old properties format, so I have left that in place as well.

@balhoff balhoff requested a review from kltm December 14, 2021 14:11
@balhoff
Copy link
Member Author

balhoff commented Dec 14, 2021

See #442.

@kltm
Copy link
Member

kltm commented Dec 15, 2021

Testing.

@kltm kltm merged commit 26fd2e1 into dev Dec 15, 2021
@balhoff balhoff deleted the log4j-xml-config branch December 16, 2021 01:57
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