----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55173/#review160845 -----------------------------------------------------------
Is this issue only for Flume? If the root cause is because of case mismatch, are other sinks (like StormTimelineMetricsSink, similar code to get hostname) prone to have the same issue? - Matt On Jan. 4, 2017, 7:23 p.m., yao lei wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55173/ > ----------------------------------------------------------- > > (Updated Jan. 4, 2017, 7:23 p.m.) > > > Review request for Ambari, Dmytro Sen, Matt, and Myroslav Papirkovskyy. > > > Bugs: AMBARI-19357 > https://issues.apache.org/jira/browse/AMBARI-19357 > > > Repository: ambari > > > Description > ------- > > If hostname of flume agent is uppercase or mixed-case?web client will not > show flume metrics normally. > This is because we will query metric records by lowercase hostname used in > ambari ,but the hostname in METRIC_RECORD is not lowercase (see attached > picture),so we will not get any records. > This patch amis to convert hostname of metric record of flume to lowercase > before inserting into ams hbase. > > > Diffs > ----- > > > ambari-metrics/ambari-metrics-flume-sink/src/main/java/org/apache/hadoop/metrics2/sink/flume/FlumeTimelineMetricsSink.java > c1b684b > > Diff: https://reviews.apache.org/r/55173/diff/ > > > Testing > ------- > > Manually Tested > > > Thanks, > > yao lei > >
