[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17280172#comment-17280172 ] Hadoop QA commented on HADOOP-15230: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Logfile || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue}{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 11s{color} | {color:red}{color} | {color:red} HADOOP-15230 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910948/HADOOP-15230.007.patch | | Console output | https://ci-hadoop.apache.org/job/PreCommit-HADOOP-Build/153/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org | This message was automatically generated. > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Assignee: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics, pull-request-available > Attachments: HADOOP-15230.007.patch > > Time Spent: 10m > Remaining Estimate: 0h > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16909670#comment-16909670 ] Hadoop QA commented on HADOOP-15230: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 9s{color} | {color:red} HADOOP-15230 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910948/HADOOP-15230.007.patch | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/16490/console | | Powered by | Apache Yetus 0.8.0 http://yetus.apache.org | This message was automatically generated. > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > Attachments: HADOOP-15230.007.patch > > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16726668#comment-16726668 ] Hadoop QA commented on HADOOP-15230: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 15s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 25m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 18m 34s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 54s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 17s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 14m 33s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 44s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 0s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 59s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 15m 20s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} javac {color} | {color:red} 15m 20s{color} | {color:red} root generated 5 new + 1485 unchanged - 5 fixed = 1490 total (was 1490) {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 52s{color} | {color:green} hadoop-common-project/hadoop-common: The patch generated 0 new + 2 unchanged - 254 fixed = 2 total (was 256) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 10s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 7s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 43s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 0s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 9m 1s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 37s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}106m 11s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:8f97d6f | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910948/HADOOP-15230.007.patch | | Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux d446ce18da98 4.4.0-139-generic #165~14.04.1-Ubuntu SMP Wed Oct 31 10:55:11 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / f659485 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_181 | | findbugs | v3.1.0-RC1 | | javac | https://builds.apache.org/job/PreCommit-HADOOP-Build/15684/artifact/out/diff-compile-javac-root.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/15684/testReport/ | | Max. process+thread count | 1382 (vs. ulimit of 1) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/156
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16409052#comment-16409052 ] ASF GitHub Bot commented on HADOOP-15230: - Github user bak12 commented on the issue: https://github.com/apache/hadoop/pull/340 😁 > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > Attachments: HADOOP-15230.007.patch > > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16369304#comment-16369304 ] ASF GitHub Bot commented on HADOOP-15230: - Github user howardyoo commented on the issue: https://github.com/apache/hadoop/pull/340 Now, the code is ready to be merged. Patch passed all the tests and without any serious conflicts. I hope the apache hadoop team would review this simple fix and merge it into the trunk branch soon. > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > Attachments: HADOOP-15230.007.patch > > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16367671#comment-16367671 ] genericqa commented on HADOOP-15230: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 18s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 18m 4s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 15m 18s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 41s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 6s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 30s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 54s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 13m 15s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 13m 15s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 38s{color} | {color:green} hadoop-common-project/hadoop-common: The patch generated 0 new + 2 unchanged - 254 fixed = 2 total (was 256) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 5s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 6s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 36s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 54s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 8m 16s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 33s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 87m 5s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910948/HADOOP-15230.007.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 9af0b7bd73a4 3.13.0-135-generic #184-Ubuntu SMP Wed Oct 18 11:55:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 82f029f | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14157/testReport/ | | Max. process+thread count | 1356 (vs. ulimit of 5500) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/14157/console | | Powered by | Apache Yetus 0.8.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > org.apache.hadoop.metrics2.GraphiteSink is no
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16366758#comment-16366758 ] genericqa commented on HADOOP-15230: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 16s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 16m 55s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 13m 19s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 4s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 2s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 45s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 12m 15s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 12m 15s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 37s{color} | {color:orange} hadoop-common-project/hadoop-common: The patch generated 9 new + 3 unchanged - 253 fixed = 12 total (was 256) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 2s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 10s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 57s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 56s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 9m 23s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 33s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 83m 58s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910882/HADOOP-15230.006.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 146f118dc091 3.13.0-135-generic #184-Ubuntu SMP Wed Oct 18 11:55:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / a1e05e0 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/14148/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14148/testReport/ | | Max. process+thread count | 1649 (vs. ulimit of 5500) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/14148/console | | Powered by
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=1630#comment-1630 ] genericqa commented on HADOOP-15230: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 15s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 16m 44s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 13m 30s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 5s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 11m 58s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 26s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 55s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 12m 17s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 12m 17s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 37s{color} | {color:orange} hadoop-common-project/hadoop-common: The patch generated 9 new + 3 unchanged - 253 fixed = 12 total (was 256) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 3s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} whitespace {color} | {color:red} 0m 0s{color} | {color:red} The patch has 6 line(s) that end in whitespace. Use git apply --whitespace=fix <>. Refer https://git-scm.com/docs/git-apply {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 21s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 39s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 53s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 9m 16s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 32s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 83m 40s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910868/HADOOP-15230.005.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 3d5952eaa9e2 3.13.0-135-generic #184-Ubuntu SMP Wed Oct 18 11:55:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 8013475 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/14146/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt | | whitespace | https://builds.apache.org/job/PreCommit-HADOOP-Build/14146/artifact/out/whitespace-eol.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14146/testReport/ | | Max. process+thread count | 1675 (vs. ulimit of 5500) | | mod
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16366580#comment-16366580 ] genericqa commented on HADOOP-15230: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 22s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 17m 50s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 16m 0s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 9s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 21s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 13m 30s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 13m 30s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 48s{color} | {color:orange} hadoop-common-project/hadoop-common: The patch generated 10 new + 126 unchanged - 130 fixed = 136 total (was 256) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 15s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} whitespace {color} | {color:red} 0m 0s{color} | {color:red} The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <>. Refer https://git-scm.com/docs/git-apply {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 29s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 35s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 56s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 8m 48s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 38s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 89m 5s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910851/HADOOP-15230.004.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux b1c2d81b99a3 3.13.0-135-generic #184-Ubuntu SMP Wed Oct 18 11:55:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 8013475 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/14144/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt | | whitespace | https://builds.apache.org/job/PreCommit-HADOOP-Build/14144/artifact/out/whitespace-eol.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14144/testReport/ | | Max. process+thread count | 1352 (vs. ulimit of 5500) | |
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16366456#comment-16366456 ] genericqa commented on HADOOP-15230: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 15s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 16m 42s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 13m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 41s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 5s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 11m 54s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 54s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 12m 17s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 12m 17s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 46s{color} | {color:orange} hadoop-common-project/hadoop-common: The patch generated 11 new + 247 unchanged - 9 fixed = 258 total (was 256) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 5s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 6s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 35s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 55s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 8m 51s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 34s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 83m 0s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910803/HADOOP-15230.003.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux b4cdbc17ff0c 3.13.0-135-generic #184-Ubuntu SMP Wed Oct 18 11:55:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 0b489e5 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/14140/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14140/testReport/ | | Max. process+thread count | 1717 (vs. ulimit of 5500) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/14140/console | | Powered b
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16366092#comment-16366092 ] genericqa commented on HADOOP-15230: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 22s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 18m 22s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 15m 2s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 41s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 7s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 11m 55s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 31s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 47s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 12m 27s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 12m 27s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 39s{color} | {color:orange} hadoop-common-project/hadoop-common: The patch generated 11 new + 247 unchanged - 9 fixed = 258 total (was 256) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 8s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} whitespace {color} | {color:red} 0m 0s{color} | {color:red} The patch has 3 line(s) that end in whitespace. Use git apply --whitespace=fix <>. Refer https://git-scm.com/docs/git-apply {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 9s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 36s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 55s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:red}-1{color} | {color:red} unit {color} | {color:red} 8m 45s{color} | {color:red} hadoop-common in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 36s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 86m 39s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.metrics2.impl.TestGraphiteMetrics | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910772/HADOOP-15230.002.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 3e4356cc4921 3.13.0-135-generic #184-Ubuntu SMP Wed Oct 18 11:55:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / da59acd | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/14135/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt | | whitespace | https://builds.apache.org/job/PreCommit-HADOOP-Build/14135/artifact/out/whitespace-eol.txt | | unit | https://builds.apache.org/job/PreCommit-HADOOP-Build/1413
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16365951#comment-16365951 ] Howard Yoo commented on HADOOP-15230: - re-submitting patch version 002 > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > Attachments: HADOOP-15230.002.patch > > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16365232#comment-16365232 ] genericqa commented on HADOOP-15230: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 15s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 15m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 12m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 38s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 0s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 11s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 15s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 55s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 39s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 11m 14s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 11m 14s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 36s{color} | {color:orange} hadoop-common-project/hadoop-common: The patch generated 7 new + 122 unchanged - 5 fixed = 129 total (was 127) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 6s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 9m 1s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 25s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 49s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:red}-1{color} | {color:red} unit {color} | {color:red} 7m 23s{color} | {color:red} hadoop-common in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 29s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 74m 49s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.metrics2.impl.TestGraphiteMetrics | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | HADOOP-15230 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12910686/HADOOP-15230.001.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 449fb820c86d 4.4.0-43-generic #63-Ubuntu SMP Wed Oct 12 13:48:03 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 8f66aff | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/14131/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt | | unit | https://builds.apache.org/job/PreCommit-HADOOP-Build/14131/artifact/out/patch-unit-hadoop-common-project_hadoop-common.txt |
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16365128#comment-16365128 ] Howard Yoo commented on HADOOP-15230: - Ok, will do. Thanks! > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16363830#comment-16363830 ] Steve Loughran commented on HADOOP-15230: - you need to hit the submit patch button to have Yetus do a build; things won't get reviewed until its happy about all the redline items, and the orangeline warnings are either addressed or justified > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16363309#comment-16363309 ] ASF GitHub Bot commented on HADOOP-15230: - GitHub user howardyoo opened a pull request: https://github.com/apache/hadoop/pull/340 fixes GraphiteSink to support point tags in correct format Fixed GraphiteSink to support point tags in correct format, having appeared at the end and also having no space as its name and value pair. [JIRA Ticket](https://issues.apache.org/jira/browse/HADOOP-15230) You can merge this pull request into a Git repository by running: $ git pull https://github.com/howardyoo/hadoop origin/HADOOP-15230 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/hadoop/pull/340.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #340 commit 70ff7d7bd562df03a86ea8819cac07606ba3b70e Author: Howard Yoo Date: 2018-02-14T01:01:02Z fixes GraphiteSink to support point tags in correct format Fixed GraphiteSink to support point tags in correct format, having appeared at the end and also having no space as its name and value pair. > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15230) org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly
[ https://issues.apache.org/jira/browse/HADOOP-15230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16363311#comment-16363311 ] Howard Yoo commented on HADOOP-15230: - pushed a new PR to fix this issue at : [https://github.com/apache/hadoop/pull/340] > org.apache.hadoop.metrics2.GraphiteSink is not implemented correctly > > > Key: HADOOP-15230 > URL: https://issues.apache.org/jira/browse/HADOOP-15230 > Project: Hadoop Common > Issue Type: Bug > Components: metrics >Reporter: Howard Yoo >Priority: Major > Labels: GraphiteSink,, metrics > > org.apache.hadoop.metrics2.GraphiteSink's implementation has certain problems > that would make it to generate metrics incorrectly. > The problem lies with line 77 ~ 84 of the GraphiteSink java: > {code:java} > for (MetricsTag tag : record.tags()) { > if (tag.value() != null) { > metricsPathPrefix.append("."); > metricsPathPrefix.append(tag.name()); > metricsPathPrefix.append("="); > metricsPathPrefix.append(tag.value()); > } > } > {code} > It produces point tags having name=value pair in the metrics. However, notice > how the tags are added with '.' as its delimiters. Rather than using the '.' > character, it should follow the following convention mentioned in the latest > graphite doc of using ';' character. > [http://graphite.readthedocs.io/en/latest/tags.html] > Also, the value is not properly being escaped, meaning that if the value has > a '.' character in it, it will easily confuse Graphite to accept it as a > delimiter, rather than the value. A really good prime example is when the > value is a hostname or ip address, > {code:java} > metrics.example.Hostname=this.is.a.hostname.and.this.is.Metrics 10.0{code} > In this example, the since the value of the hostname contains '.', it is > extremely hard for the receiving end to determine which part is hostname and > which part is the rest of the metrics name. A good strategy is to convert any > '.' character in the value to be converted to other characters, such as '_'. > However, the best way would be to follow the latest metrics convention of > using ';' > {code:java} > metrics.example.and.this.is.Metrics;Hostname=this.is.a.hostname 10.0{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org