steveloughran commented on issue #1186: HADOOP-16472. findbugs warning on LocalMetadataStore.ttlTimeProvider sync URL: https://github.com/apache/hadoop/pull/1186#issuecomment-517849769 That's the same sync problem; I'm going to merge this one in. I don't care about a line length of 81; we are allowed to ignore that 81 char width rule if we think it would be worse for the code visibility to chop things...the last time there was a vote on it the consensus was "not widen because it makes side-by-side review harder". Once the move to github reviews is complete, we can explore that decision again. After all, monitors have got bigger...
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
