From 5596e6061115307dcbf5b34c4181b201c103fc72 Mon Sep 17 00:00:00 2001 From: Ivan Topolnak Date: Wed, 21 May 2014 18:17:30 -0300 Subject: = core: avoid using tuples in favor of case classes --- .../scala/kamon/metrics/DispatcherMetrics.scala | 4 +- .../dispatcher/DispatcherMetricsCollector.scala | 59 ---------------------- .../kamon/metrics/instruments/MinMaxCounter.scala | 55 ++++++++++++++++++++ .../instruments/counter/MinMaxCounter.scala | 51 ------------------- 4 files changed, 57 insertions(+), 112 deletions(-) delete mode 100644 kamon-core/src/main/scala/kamon/metrics/dispatcher/DispatcherMetricsCollector.scala create mode 100644 kamon-core/src/main/scala/kamon/metrics/instruments/MinMaxCounter.scala delete mode 100644 kamon-core/src/main/scala/kamon/metrics/instruments/counter/MinMaxCounter.scala (limited to 'kamon-core/src/main/scala/kamon') diff --git a/kamon-core/src/main/scala/kamon/metrics/DispatcherMetrics.scala b/kamon-core/src/main/scala/kamon/metrics/DispatcherMetrics.scala index 77a1c99b..7e9198f3 100644 --- a/kamon-core/src/main/scala/kamon/metrics/DispatcherMetrics.scala +++ b/kamon-core/src/main/scala/kamon/metrics/DispatcherMetrics.scala @@ -26,7 +26,7 @@ case class DispatcherMetrics(name: String) extends MetricGroupIdentity { object DispatcherMetrics extends MetricGroupCategory { val name = "dispatcher" - case object MaximunPoolSize extends MetricIdentity { val name, tag = "maximum-pool-size" } + case object MaximumPoolSize extends MetricIdentity { val name, tag = "maximum-pool-size" } case object RunningThreadCount extends MetricIdentity { val name, tag = "running-thread-count" } case object QueueTaskCount extends MetricIdentity { val name, tag = "queued-task-count" } case object PoolSize extends MetricIdentity { val name, tag = "pool-size" } @@ -43,7 +43,7 @@ object DispatcherMetrics extends MetricGroupCategory { extends MetricGroupSnapshot { val metrics: Map[MetricIdentity, MetricSnapshotLike] = Map( - (MaximunPoolSize -> maximumPoolSize), + (MaximumPoolSize -> maximumPoolSize), (RunningThreadCount -> runningThreadCount), (QueueTaskCount -> queueTaskCount), (PoolSize -> poolSize)) diff --git a/kamon-core/src/main/scala/kamon/metrics/dispatcher/DispatcherMetricsCollector.scala b/kamon-core/src/main/scala/kamon/metrics/dispatcher/DispatcherMetricsCollector.scala deleted file mode 100644 index 529cc542..00000000 --- a/kamon-core/src/main/scala/kamon/metrics/dispatcher/DispatcherMetricsCollector.scala +++ /dev/null @@ -1,59 +0,0 @@ -package kamon.metrics.dispatcher - -/* - * ========================================================================================= - * Copyright © 2013-2014 the kamon project - * - * 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 - * - * 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. - * ========================================================================================= - */ - -import scala.concurrent.forkjoin.ForkJoinPool -import java.util.concurrent.ThreadPoolExecutor -import akka.dispatch.{ ExecutorServiceDelegate, Dispatcher, MessageDispatcher } -import java.lang.reflect.Method - -object DispatcherMetricsCollector { - - private[this]type DispatcherMetrics = (Long, Long, Long, Long) - - private[this] def collectForkJoinMetrics(pool: ForkJoinPool): DispatcherMetrics = { - (pool.getParallelism, pool.getActiveThreadCount, (pool.getQueuedTaskCount + pool.getQueuedSubmissionCount), pool.getPoolSize) - } - private[this] def collectExecutorMetrics(pool: ThreadPoolExecutor): DispatcherMetrics = { - (pool.getMaximumPoolSize, pool.getActiveCount, pool.getQueue.size(), pool.getPoolSize) - } - - private[this] val executorServiceMethod: Method = { - // executorService is protected - val method = classOf[Dispatcher].getDeclaredMethod("executorService") - method.setAccessible(true) - method - } - - def collect(dispatcher: MessageDispatcher): (Long, Long, Long, Long) = { - dispatcher match { - case x: Dispatcher ⇒ { - val executor = executorServiceMethod.invoke(x) match { - case delegate: ExecutorServiceDelegate ⇒ delegate.executor - case other ⇒ other - } - - executor match { - case fjp: ForkJoinPool ⇒ collectForkJoinMetrics(fjp) - case tpe: ThreadPoolExecutor ⇒ collectExecutorMetrics(tpe) - case anything ⇒ (0L, 0L, 0L, 0L) - } - } - case _ ⇒ new DispatcherMetrics(0L, 0L, 0L, 0L) - } - } -} diff --git a/kamon-core/src/main/scala/kamon/metrics/instruments/MinMaxCounter.scala b/kamon-core/src/main/scala/kamon/metrics/instruments/MinMaxCounter.scala new file mode 100644 index 00000000..7f6c2487 --- /dev/null +++ b/kamon-core/src/main/scala/kamon/metrics/instruments/MinMaxCounter.scala @@ -0,0 +1,55 @@ +package kamon.metrics.instruments + +/* + * ========================================================================================= + * Copyright © 2013-2014 the kamon project + * + * 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 + * + * 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. + * ========================================================================================= + */ + +import java.lang.Math._ +import jsr166e.LongMaxUpdater +import kamon.util.PaddedAtomicLong +import kamon.metrics.instruments.MinMaxCounter.CounterMeasurement + +class MinMaxCounter { + private val min = new LongMaxUpdater + private val max = new LongMaxUpdater + private val sum = new PaddedAtomicLong + + min.update(0L) + max.update(0L) + + def increment(value: Long = 1L): Unit = { + val currentValue = sum.addAndGet(value) + max.update(currentValue) + } + + def decrement(value: Long = 1L): Unit = { + val currentValue = sum.addAndGet(-value) + min.update(-currentValue) + } + + def collect(): CounterMeasurement = { + val currentValue = sum.get() + val result = CounterMeasurement(abs(min.maxThenReset()), max.maxThenReset(), currentValue) + max.update(currentValue) + min.update(-currentValue) + result + } +} + +object MinMaxCounter { + def apply() = new MinMaxCounter() + + case class CounterMeasurement(min: Long, max: Long, current: Long) +} diff --git a/kamon-core/src/main/scala/kamon/metrics/instruments/counter/MinMaxCounter.scala b/kamon-core/src/main/scala/kamon/metrics/instruments/counter/MinMaxCounter.scala deleted file mode 100644 index 6c89a4d1..00000000 --- a/kamon-core/src/main/scala/kamon/metrics/instruments/counter/MinMaxCounter.scala +++ /dev/null @@ -1,51 +0,0 @@ -package kamon.metrics.instruments.counter -/* - * ========================================================================================= - * Copyright © 2013-2014 the kamon project - * - * 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 - * - * 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. - * ========================================================================================= - */ - -import java.lang.Math._ -import jsr166e.LongMaxUpdater -import kamon.util.PaddedAtomicLong - -class MinMaxCounter { - private val min = new LongMaxUpdater - private val max = new LongMaxUpdater - private val sum = new PaddedAtomicLong - - min.update(0L) - max.update(0L) - - def increment(value: Long = 1L): Unit = { - val currentValue = sum.addAndGet(value) - max.update(currentValue) - } - - def decrement(value: Long = 1L): Unit = { - val currentValue = sum.addAndGet(-value) - min.update(-currentValue) - } - - def collect(): (Long, Long, Long) = { - val currentValue = sum.get() - val result = (abs(min.maxThenReset()), max.maxThenReset(), currentValue) - max.update(currentValue) - min.update(-currentValue) - result - } -} - -object MinMaxCounter { - def apply() = new MinMaxCounter() -} -- cgit v1.2.3