Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Plain View
MapReduce >> mail # dev >> API Design: getClusterMetrics


+
Saptarshi Guha 2012-10-10, 04:11
Copy link to this message
-
Re: API Design: getClusterMetrics

This is to help evolve the APIs easily. All apis have request and response objects, instead of passing in parameters directly. Some of the APIs clearly may not need this extra layer, but doing so helps uniformity.

Irrespective of that, following the original goal, GetClusterMetricsResponse itself could have all the fields, instead of one more nested ClusterMetrics object.

Thanks,
+Vinod

On Oct 9, 2012, at 9:11 PM, Saptarshi Guha wrote:

> Hello
> (sorry for cross post)
> I am reading the DistributedShell code and came across
>
> GetClusterMetricsRequest clusterMetricsReq > Records.newRecord(GetClusterMetricsRequest.class);
> GetClusterMetricsResponse clusterMetricsResp > applicationsManager.getClusterMetrics(clusterMetricsReq);
>
> In the page: http://hadoop.apache.org/docs/current/api/org/apache/hadoop/yarn/api/ClientRMProtocol.html#getClusterMetrics%28org.apache.hadoop.yarn.api.protocolrecords.GetClusterMetricsRequest%29
>
> I can't find any other function getClusterMetrics.
>
> Why was the choice made to pass a GetClusterMetricsRequest object to
> getClusterMetrics?
> why couldn't the programmer invoke
> applicationsManager.getClusterMetrics() and the method would reply
> with a GetClusterMetricsResponse?
>
> Regards
> Sapsi
> (P.S I havent used YARN or even written code with it, i'm just reading
> some sources and my reading is incomplete, so apologies if this comes
> across
> as stupid)

+
Robert Evans 2012-10-10, 14:30
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB