Skip to content

Factor out glog references for ease of dropping in an adapter to a di…#3855

Merged
michael-berlin merged 1 commit intovitessio:masterfrom
michael-berlin:restore_glog_change
Apr 21, 2018
Merged

Factor out glog references for ease of dropping in an adapter to a di…#3855
michael-berlin merged 1 commit intovitessio:masterfrom
michael-berlin:restore_glog_change

Conversation

@michael-berlin
Copy link
Copy Markdown
Contributor

@michael-berlin michael-berlin commented Apr 21, 2018

…fferent logging library.

Note: This is the revert of a temporary revert and has the same effect
as the original PR: #3830

Revert "Revert "Factor out glog references""

This reverts commit 246e289.

Signed-off-by: David Weitzman dweitzman@pinterest.com

…fferent logging library.

Note: This is the revert of a temporary revert and has the same effect
as the original PR: vitessio#3830

Revert "Revert "Factor out glog references""

This reverts commit 246e289.

Signed-off-by: David Weitzman <dweitzman@pinterest.com>
@arthurnn
Copy link
Copy Markdown
Contributor

❤️

@vmg and I had issues with the vitess logger before, this is a good step IMO

@michael-berlin
Copy link
Copy Markdown
Contributor Author

Credit goes to David Weitzman.

I am just re-merging this after I had to temporarily revert the original PR earlier.

@michael-berlin michael-berlin merged commit 210495b into vitessio:master Apr 21, 2018
@michael-berlin michael-berlin deleted the restore_glog_change branch April 21, 2018 03:28
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.

4 participants