reef-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (REEF-1732) Build Metrics System
Date Fri, 15 Jun 2018 23:26:01 GMT

    [ https://issues.apache.org/jira/browse/REEF-1732?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16514502#comment-16514502
] 

ASF GitHub Bot commented on REEF-1732:
--------------------------------------

singlis commented on a change in pull request #1460: [REEF-1732] Build Metrics System
URL: https://github.com/apache/reef/pull/1460#discussion_r195882450
 
 

 ##########
 File path: lang/cs/Org.Apache.REEF.Common/Telemetry/MetricBase.cs
 ##########
 @@ -0,0 +1,114 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The ASF licenses this file
+// to you 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.
+
+using System;
+using System.Threading;
+using Newtonsoft.Json;
+
+namespace Org.Apache.REEF.Common.Telemetry
+{
+    /// <summary>
+    /// Base implementation of a metric object.
+    /// </summary>
+    /// <typeparam name="T">Metric type</typeparam>
+    public class MetricBase<T> : IMetric<T>
+    {
+        protected ITracker _tracker;
+
+        protected T _typedValue;
+
+        protected bool _isImmutable;
+
+        private object _metricLock = new object();
+
+        public string Name { get; }
+
+        public string Description { get; }
+
+        public virtual object ValueUntyped
+        {
+            get { return _typedValue; }
+        }
+
+        public virtual T Value
+        {
+            get
+            {
+                return _typedValue;
+            }
+        }
+
+        public bool IsImmutable
+        {
+            get
+            {
+                return _isImmutable;
+            }
+        }
+
+        public MetricBase(string name, string description, bool isImmutable = true)
+        {
+            Name = name;
+            Description = description;
+            _typedValue = default(T);
+            _isImmutable = isImmutable;
+        }
+
+        [JsonConstructor]
+        public MetricBase(string name, string description, T value)
+        {
+            Name = name;
+            Description = description;
+            _typedValue = value;
 
 Review comment:
   Is isImmutable needed here? Should it be set to a default value?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Build Metrics System
> --------------------
>
>                 Key: REEF-1732
>                 URL: https://issues.apache.org/jira/browse/REEF-1732
>             Project: REEF
>          Issue Type: New Feature
>          Components: IMRU, REEF
>            Reporter: Julia
>            Assignee: Julia
>            Priority: Major
>         Attachments: IMRU Metrics System.docx
>
>
> IMRU Metrics is to provide metrics data to the system so that it can be shown to the
user for monitoring or diagnosis. The goal is to build an E2E flow with simple/basic metrics
data. We can then add more data later. 
> * IMetricsProvider - there are multiple sources of metrics data:
>   1.Task metrics. This is in particular for IMRU task such as current iteration, progress.
Each task can send task state back to driver and let driver to aggregate it. Alternatively,
as UpdateTask knows current iterations and progress, to start with, we can just get task status
from update task. The task metrics can be provided by task function like IUpdateFunction and
send to driver by task host as TaskMessage with heartbeat. 
>   2. Driver metrics – For IMRU driver, it can be system state such as WaitingForEvaluator
or TasksRunning, current retry number, etc. Those driver states are maintained inside IMRU
driver. 
>  3. IMRUDriver will implement IMetricsProvider and supply metrics data. 
> * IMetricsSink – the metrics data will be output somewhere so that it can be consumed
by a monitoring tool. An interface IMetricsSink will be defined to sink metrics data. An implementation
of the interface can store the data to a remote storage. Multiple sinks can be injected. 
> * MetricsManager – It schedules a timer to get metrics from IMetricsProviders and output
the metrics data with IMetricsSinks
> Attached file shows the diagram of the design. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message