[incubator-pinot] branch introduce-factory-for-metrics updated (e01a601 -> 0231487)

[jl...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jlli pushed a change to branch introduce-factory-for-metrics
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.


    from e01a601  Decouple from yammer library
     add 0231487  Address PR comments

No new revisions were added by this update.

Summary of changes:
 .../broker/broker/helix/HelixBrokerStarter.java    | 10 +--
 .../LiteralOnlyBrokerRequestTest.java              |  6 +-
 .../pinot/common/metrics/AbstractMetrics.java      | 19 ++---
 .../apache/pinot/common/metrics/BrokerMetrics.java |  2 +-
 .../pinot/common/metrics/ControllerMetrics.java    |  2 +-
 ...eporterMetricsRegistryRegistrationListener.java |  5 +-
 .../apache/pinot/common/metrics/LatencyMetric.java | 95 ----------------------
 .../apache/pinot/common/metrics/MetricsHelper.java | 14 ++--
 .../MetricsRegistryRegistrationListener.java       |  2 +-
 ...tricUtilsFactory.java => PinotMetricUtils.java} | 10 ++-
 .../apache/pinot/common/metrics/ServerMetrics.java |  2 +-
 .../pinot/common/metrics/ValidationMetrics.java    | 15 ++--
 .../pinot/common/metrics/yammer/YammerCounter.java |  6 +-
 .../pinot/common/metrics/yammer/YammerGauge.java   |  6 +-
 .../common/metrics/yammer/YammerJmxReporter.java   |  4 +-
 .../pinot/common/metrics/yammer/YammerMeter.java   |  6 +-
 .../pinot/common/metrics/yammer/YammerMetered.java |  2 +-
 .../pinot/common/metrics/yammer/YammerMetric.java  |  6 +-
 .../common/metrics/yammer/YammerMetricName.java    |  2 +-
 .../metrics/yammer/YammerMetricProcessor.java      | 14 ++--
 .../metrics/yammer/YammerMetricsRegistry.java      | 16 ++--
 .../pinot/common/metrics/yammer/YammerTimer.java   |  6 +-
 .../pinot/common/metrics/MetricsHelperTest.java    |  9 +-
 ...sFactoryTest.java => PinotMetricUtilsTest.java} | 13 ++-
 .../spark/connector/PinotServerDataFetcher.scala   |  5 +-
 .../apache/pinot/controller/ControllerStarter.java | 10 +--
 .../core/periodictask/ControllerPeriodicTask.java  |  1 -
 .../controller/LeadControllerManagerTest.java      |  4 +-
 .../pinot/controller/api/TableSizeReaderTest.java  |  4 +-
 .../controller/helix/SegmentStatusCheckerTest.java | 24 +++---
 .../periodictask/ControllerPeriodicTaskTest.java   |  4 +-
 .../helix/core/realtime/SegmentCompletionTest.java |  6 +-
 .../helix/core/retention/RetentionManagerTest.java |  6 +-
 .../core/retention/SegmentLineageCleanupTest.java  |  4 +-
 .../validation/StorageQuotaCheckerTest.java        |  4 +-
 .../realtime/HLRealtimeSegmentDataManager.java     |  2 +-
 .../realtime/LLRealtimeSegmentDataManager.java     |  2 +-
 .../core/io/writer/impl/DirectMemoryManager.java   |  4 +-
 .../core/io/writer/impl/MmapMemoryManager.java     |  4 +-
 .../data/manager/BaseTableDataManagerTest.java     |  4 +-
 .../offline/DimensionTableDataManagerTest.java     |  4 +-
 .../realtime/LLRealtimeSegmentDataManagerTest.java |  6 +-
 .../scheduler/MultiLevelPriorityQueueTest.java     |  4 +-
 .../query/scheduler/PrioritySchedulerTest.java     |  4 +-
 .../scheduler/fcfs/FCFSSchedulerGroupTest.java     |  4 +-
 .../SegmentGenerationWithNullValueVectorTest.java  |  4 +-
 .../pinot/query/executor/QueryExecutorTest.java    |  4 +-
 .../tests/SegmentCompletionIntegrationTest.java    |  4 +-
 .../org/apache/pinot/minion/MinionStarter.java     | 22 ++---
 .../apache/pinot/minion/metrics/MinionMetrics.java |  2 +-
 .../pinot/server/starter/ServerInstance.java       | 10 +--
 .../apache/pinot/spi/metrics}/PinotCounter.java    |  2 +-
 .../org/apache/pinot/spi/metrics}/PinotGauge.java  |  2 +-
 .../apache/pinot/spi/metrics}/PinotHistogram.java  |  2 +-
 .../pinot/spi/metrics}/PinotJmxReporter.java       |  2 +-
 .../org/apache/pinot/spi/metrics}/PinotMeter.java  |  2 +-
 .../apache/pinot/spi/metrics}/PinotMetered.java    |  2 +-
 .../org/apache/pinot/spi/metrics}/PinotMetric.java |  2 +-
 .../apache/pinot/spi/metrics}/PinotMetricName.java |  2 +-
 .../pinot/spi/metrics}/PinotMetricProcessor.java   |  2 +-
 .../pinot/spi/metrics}/PinotMetricsRegistry.java   |  2 +-
 .../org/apache/pinot/spi/metrics}/PinotTimer.java  |  2 +-
 62 files changed, 171 insertions(+), 279 deletions(-)
 delete mode 100644 pinot-common/src/main/java/org/apache/pinot/common/metrics/LatencyMetric.java
 rename pinot-common/src/main/java/org/apache/pinot/common/metrics/{base/PinotMetricUtilsFactory.java => PinotMetricUtils.java} (90%)
 rename pinot-common/src/test/java/org/apache/pinot/common/metrics/{PinotMetricUtilsFactoryTest.java => PinotMetricUtilsTest.java} (81%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotCounter.java (95%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotGauge.java (95%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotHistogram.java (95%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotJmxReporter.java (94%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotMeter.java (95%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotMetered.java (98%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotMetric.java (96%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotMetricName.java (95%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotMetricProcessor.java (98%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotMetricsRegistry.java (97%)
 rename {pinot-common/src/main/java/org/apache/pinot/common/metrics/base => pinot-spi/src/main/java/org/apache/pinot/spi/metrics}/PinotTimer.java (95%)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org