Skip to content

Conversation

@rfarivar
Copy link
Contributor

This is to reduce the noisiness of supervisor logs.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to use the parameterized call, so that Strings are not constructed unnecessarily:

Log.debug("The baseSleepTimeMs [{}] ... ", baseSleepTimeMs, ...);

@knusbaum
Copy link
Contributor

+1

3 similar comments
@HeartSaVioR
Copy link
Contributor

+1

@nathanmarz
Copy link
Contributor

+1

@revans2
Copy link
Contributor

revans2 commented Jul 15, 2015

+1

@asfgit asfgit merged commit 1b3e9ed into apache:master Jul 15, 2015
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.

7 participants