Hadoop Common
  1. Hadoop Common
  2. HADOOP-7330

The metrics source mbean implementation should return the attribute value instead of the object

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.20.203.0
    • Fix Version/s: 0.20.203.1, 0.20.204.0
    • Component/s: metrics
    • Labels:
      None
    • Tags:
      metrics, jmx

      Description

      The MetricsSourceAdapter#getAttribute in 0.20.203 is returning the attribute object instead of the value.

        Activity

        Hide
        Luke Lu added a comment -

        This is only for the branch-0.20-security* branches. yahoo-merge and trunk do not have this bug.

        Show
        Luke Lu added a comment - This is only for the branch-0.20-security* branches. yahoo-merge and trunk do not have this bug.
        Hide
        Robert Joseph Evans added a comment -

        Comment +1 on the patch. It looks good, some tests would be nice, but it is a single line change that really doesn't impact much. Plus it makes 0.20.20X consistent with trunk.

        Show
        Robert Joseph Evans added a comment - Comment +1 on the patch. It looks good, some tests would be nice, but it is a single line change that really doesn't impact much. Plus it makes 0.20.20X consistent with trunk.
        Hide
        Luke Lu added a comment -

        some tests would be nice.

        Agreed. It'd be awesome if you can take up HADOOP-7273, which would help us write tests for JMX stuff outside HDFS.

        Show
        Luke Lu added a comment - some tests would be nice. Agreed. It'd be awesome if you can take up HADOOP-7273 , which would help us write tests for JMX stuff outside HDFS.
        Hide
        Owen O'Malley added a comment -

        I just committed this. Thanks, Luke!

        Show
        Owen O'Malley added a comment - I just committed this. Thanks, Luke!
        Hide
        Owen O'Malley added a comment -

        Hadoop 0.20.204.0 was released today.

        Show
        Owen O'Malley added a comment - Hadoop 0.20.204.0 was released today.

          People

          • Assignee:
            Luke Lu
            Reporter:
            Luke Lu
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development