> On 一月 9, 2017, 1:23 a.m., Matt wrote: > > 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?
Hi Matt. Thanks for your reply. I am not sure whether other service have same issue.I happened to click one host of flume agents in Flume Summary page and found its Channel Metrics show nothing. This flume agent actual hostname is GG3 and displayed in Flume Summary and Hosts page as gg3 In web browser I found some requests sent to server like, such as http://gg5:8080/api/v1/clusters/test/hosts/gg3/host_components/FLUME_HANDLER?fields=metrics/flume/flume/CHANNEL/*&_=1483927924281 http://gg5:8080/api/v1/clusters/test/hosts/gg3/host_components/FLUME_HANDLER?fields=metrics/flume/flume/CHANNEL/ch2/ChannelCapacity[1483924529,1483928129,15]&_=1483927924545 These request hostname is lowercase gg3 and will not match the value GG3 stored in AMS table METRIC_RECORD. - yao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55173/#review160845 ----------------------------------------------------------- On 一月 5, 2017, 3:23 a.m., yao lei wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55173/ > ----------------------------------------------------------- > > (Updated 一月 5, 2017, 3:23 a.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 > >
