[ https://issues.apache.org/jira/browse/YARN-3862?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15035275#comment-15035275
]
Varun Saxena commented on YARN-3862:
------------------------------------
[~sjlee0], kindly review. Regarding some of the checkstyle comments.
If below two checkstyle issues have to be fixed, I think they should be fixed for around 10
other variables as well. We are directly accessing these variables in subclasses(***EntityReader).
Do you want me to add getters and setters for each one of them ?
{noformat}
./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice/src/main/java/org/apache/hadoop/yarn/server/timelineservice/storage/TimelineEntityReader.java:75:32:
Variable 'confsToRetrieve' must be private and have accessor methods.
./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice/src/main/java/org/apache/hadoop/yarn/server/timelineservice/storage/TimelineEntityReader.java:76:32:
Variable 'metricsToRetrieve' must be private and have accessor methods.
{noformat}
Below checkstyle issue is introduced because of javadoc. This import is required in javadoc
as I have a link to this class in javadoc. If I give full classname along with package name,
it threw multiple javadoc warnings in previous build. So I guess this should be fine.
{noformat}
./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice/src/main/java/org/apache/hadoop/yarn/server/timelineservice/storage/TimelineReader.java:31:8:
Unused import - org.apache.hadoop.yarn.server.timelineservice.reader.filter.TimelinePrefixFilter.
{noformat}
> Support for fetching specific configs and metrics based on prefixes
> -------------------------------------------------------------------
>
> Key: YARN-3862
> URL: https://issues.apache.org/jira/browse/YARN-3862
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: timelineserver
> Affects Versions: YARN-2928
> Reporter: Varun Saxena
> Assignee: Varun Saxena
> Labels: yarn-2928-1st-milestone
> Attachments: YARN-3862-YARN-2928.wip.01.patch, YARN-3862-YARN-2928.wip.02.patch,
YARN-3862-feature-YARN-2928.005.patch, YARN-3862-feature-YARN-2928.04.patch, YARN-3862-feature-YARN-2928.wip.03.patch
>
>
> Currently, we will retrieve all the contents of the field if that field is specified
in the query API. In case of configs and metrics, this can become a lot of data even though
the user doesn't need it. So we need to provide a way to query only a set of configs or metrics.
> As a comma spearated list of configs/metrics to be returned will be quite cumbersome
to specify, we have to support either of the following options :
> # Prefix match
> # Regex
> # Group the configs/metrics and query that group.
> We also need a facility to specify a metric time window to return metrics in a that window.
This may be useful in plotting graphs
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
|