Support option to use Slf4j/Commons Logging/JUL when using the command line tool #1075

Closed
ericparton opened this Issue Aug 9, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@ericparton

Currently, when using the command line tool, Flyway only supports logging to standard out. It would be useful if the command line tool could optionally leverage Flyway's existing ability to use Slf4j or Commons Logging if their jars are on the classpath.

@axelfontaine

This comment has been minimized.

Show comment
Hide comment
@axelfontaine

axelfontaine Aug 12, 2015

Contributor

Thanks for your suggestion. What would be the real world use case for this?

Cheers
Axel

Contributor

axelfontaine commented Aug 12, 2015

Thanks for your suggestion. What would be the real world use case for this?

Cheers
Axel

@ericparton

This comment has been minimized.

Show comment
Hide comment
@ericparton

ericparton Aug 17, 2015

It would allow users of the command line tool to log the output of the tool to a file on disk if a jar with a logging configuration file and the logging jars were on the classpath. You could also do things like configure the logger to print the info statements to the console, but only log the debug statements to a file, so you don't have to run the tool with the debug flag and you still have the ability to consult the logs if there's an issue.

Thanks,
Eric

It would allow users of the command line tool to log the output of the tool to a file on disk if a jar with a logging configuration file and the logging jars were on the classpath. You could also do things like configure the logger to print the info statements to the console, but only log the debug statements to a file, so you don't have to run the tool with the debug flag and you still have the ability to consult the logs if there's an issue.

Thanks,
Eric

axelfontaine added a commit to flyway/flywaydb.org that referenced this issue Oct 1, 2015

@axelfontaine

This comment has been minimized.

Show comment
Hide comment
@axelfontaine

axelfontaine Oct 1, 2015

Contributor

I ended up implementing it with an automatic switch instead.

Cheers
Axel

Contributor

axelfontaine commented Oct 1, 2015

I ended up implementing it with an automatic switch instead.

Cheers
Axel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment