Remove usage of LazyLogging

[echeipesh]

Overview

GeoTrellis uses com.typesafe.scalalogging package. This is a popular library but unfortunately the LazyLogging trait that is used to mix-in a logger has shifted around a number of timers through the version history. This causes frequent binary incompatibility problems.

This PR removes any mention of LazyLogging trait and replaces it with protected lazy val logger: Logger fields. The Logger class is a little more stable.

Additionally:

• ArrayTile no longer warns on narrowing celltype conversion
• GeoWave/GeoMesa do not log errors with experimental notice

Checklist

• docs/CHANGELOG.rst updated, if necessary
• [ ] docs guides update, if necessary
• [ ] New user API has useful Scaladoc strings
• [ ] Unit tests added for bug-fix or new feature

[pomadchin]

Hm, I like that we had to extend a common LazyLogger trait, sounds like a much better alternative to copy-pasting everywhere the same @transient protected lazy val logger = Logger(LoggerFactory.getLogger(getClass.getName)) line.

[echeipesh]

Do you feel strongly about that? I think I'm being just a bit defensive and trying to make logging as private as possible without even dependency on geotrellis-util package

[metasim]

Thanks guys for this!! I'm also in the defensive camp, even if it's not rational.

[pomadchin]

@metasim we came to a solution that satisfied both sides #3116