diff options
author | Diego <diegolparra@gmail.com> | 2014-10-18 23:57:26 -0300 |
---|---|---|
committer | Diego <diegolparra@gmail.com> | 2014-10-18 23:57:26 -0300 |
commit | fc22c55f1c5caac4a4921855c30b966722ec8157 (patch) | |
tree | b3791628007315c645e05c2974940c2b08c5b3bd /kamon-newrelic/src/main/scala/kamon/newrelic/NewRelicErrorLogger.scala | |
parent | 0f4847445b31a2a76897f7405512a58fb4d4a1dd (diff) | |
download | Kamon-fc22c55f1c5caac4a4921855c30b966722ec8157.tar.gz Kamon-fc22c55f1c5caac4a4921855c30b966722ec8157.tar.bz2 Kamon-fc22c55f1c5caac4a4921855c30b966722ec8157.zip |
! kamon-newrelic: * Avoid reporting data to Newrelic if no metrics have been collected
* Implement error handling with NewRelic Agent * Minor refactor * close issue #7 and issue #17
Diffstat (limited to 'kamon-newrelic/src/main/scala/kamon/newrelic/NewRelicErrorLogger.scala')
-rw-r--r-- | kamon-newrelic/src/main/scala/kamon/newrelic/NewRelicErrorLogger.scala | 42 |
1 files changed, 18 insertions, 24 deletions
diff --git a/kamon-newrelic/src/main/scala/kamon/newrelic/NewRelicErrorLogger.scala b/kamon-newrelic/src/main/scala/kamon/newrelic/NewRelicErrorLogger.scala index 4203f81f..4bb0ad3a 100644 --- a/kamon-newrelic/src/main/scala/kamon/newrelic/NewRelicErrorLogger.scala +++ b/kamon-newrelic/src/main/scala/kamon/newrelic/NewRelicErrorLogger.scala @@ -1,23 +1,23 @@ -/* =================================================== - * Copyright © 2013 the kamon project <http://kamon.io/> +/* + * ========================================================================================= + * Copyright © 2013-2014 the kamon project <http://kamon.io/> * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at + * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file + * except in compliance with the License. You may obtain a copy of the License at * - * http://www.apache.org/licenses/LICENSE-2.0 + * http://www.apache.org/licenses/LICENSE-2.0 * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - * ========================================================== */ + * Unless required by applicable law or agreed to in writing, software distributed under the + * License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, + * either express or implied. See the License for the specific language governing permissions + * and limitations under the License. + * ========================================================================================= + */ + package kamon.newrelic -import akka.actor.{ ActorLogging, Actor } -import akka.event.Logging.Error -import akka.event.Logging.{ LoggerInitialized, InitializeLogger } +import akka.actor.{ Actor, ActorLogging } +import akka.event.Logging.{ Error, InitializeLogger, LoggerInitialized } import com.newrelic.api.agent.{ NewRelic ⇒ NR } import kamon.trace.TraceContextAware @@ -33,16 +33,10 @@ class NewRelicErrorLogger extends Actor with ActorLogging { def notifyError(error: Error): Unit = { val params = new java.util.HashMap[String, String]() - if (error.isInstanceOf[TraceContextAware]) { - val ctx = error.asInstanceOf[TraceContextAware].traceContext + val ctx = error.asInstanceOf[TraceContextAware].traceContext - for (c ← ctx) { - params.put("TraceToken", c.token) - } - } else if (!aspectJMissingAlreadyReported) { - log.warning("ASPECTJ WEAVER MISSING. You might have missed to include the javaagent JVM startup parameter in" + - " your application. Please refer to http://kamon.io/get-started/ for instructions on how to do it.") - aspectJMissingAlreadyReported = true + for (c ← ctx) { + params.put("TraceToken", c.token) } if (error.cause == Error.NoCause) { |