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

Fix logging of etc directory location #395

Merged
merged 1 commit into from Mar 6, 2019

Conversation

2 participants
@findepi
Copy link
Member

findepi commented Mar 6, 2019

Logging destination is set up in
io.airlift.bootstrap.Bootstrap#initialize (by calling
io.airlift.log.Logging#configure). Before that point, logging goes to
launcher.log which is rarely consulted.

Fix logging of etc directory location
Logging destination is set up in
`io.airlift.bootstrap.Bootstrap#initialize` (by calling
`io.airlift.log.Logging#configure`). Before that point, logging goes to
launcher.log which is rarely consulted.

@cla-bot cla-bot bot added the cla-signed label Mar 6, 2019

@findepi findepi requested a review from kokosing Mar 6, 2019

@findepi

This comment has been minimized.

Copy link
Member Author

findepi commented Mar 6, 2019

@kokosing i have to undo #365 (comment) suggestion
unfortunately i didn't check the consequences 😞

@dain

dain approved these changes Mar 6, 2019

@findepi findepi merged commit 18d7595 into prestosql:master Mar 6, 2019

1 check passed

verification/cla-signed
Details

@findepi findepi deleted the findepi:findepi/master/fix-logging-of-etc-directory-location-91e7c5 branch Mar 6, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.