HBase
  1. HBase
  2. HBASE-5598

Analyse and fix the findbugs reporting by QA and add invalid bugs into findbugs-excludeFilter file

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 0.92.1, 0.94.0, 0.95.2
    • Fix Version/s: 0.95.0
    • Component/s: scripts
    • Labels:
      None
    • Release Note:
      Hide
      Developers can generate the findbugs report for analysing the findbugs violations.
      Execute this maven command for running and generating the findbugs report: mvn findbugs:gui
      This generates the report in GUI, by which will directly show the code instances where exactly the issue is reported.

      If the developer finds any invalid bug, then that bug pattern can be updated in dev-support/findbugs-exclude.xml file.
      example:
      <Match>
         <Class name="org.apache.hadoop.hbase.regionserver.HRegion" />
         <Method name="getRecentFlushInfo" />
         <Bug pattern="UL_UNRELEASED_LOCK_EXCEPTION_PATH" />
      </Match>
      Show
      Developers can generate the findbugs report for analysing the findbugs violations. Execute this maven command for running and generating the findbugs report: mvn findbugs:gui This generates the report in GUI, by which will directly show the code instances where exactly the issue is reported. If the developer finds any invalid bug, then that bug pattern can be updated in dev-support/findbugs-exclude.xml file. example: <Match>    <Class name="org.apache.hadoop.hbase.regionserver.HRegion" />    <Method name="getRecentFlushInfo" />    <Bug pattern="UL_UNRELEASED_LOCK_EXCEPTION_PATH" /> </Match>
    • Tags:
      noob

      Description

      There are many findbugs errors reporting by HbaseQA. HBASE-5597 is going to up the OK count.
      This may lead to other issues when we re-factor the code, if we induce new valid ones and remove invalid bugs also can not be reported by QA.

      So, I would propose to add the exclude filter file for findbugs(for the invalid bugs). If we find any valid ones, we can fix under this JIRA.

      1. 5598.part1.patch
        23 kB
        Nicolas Liochon
      2. 5598.part2.patch
        24 kB
        Nicolas Liochon
      3. 5598.part2.patch
        24 kB
        Nicolas Liochon
      4. ASF.LICENSE.NOT.GRANTED--findbugs-gui-report.jpg
        175 kB
        Uma Maheswara Rao G
      5. HBASE-5598.patch
        2 kB
        Uma Maheswara Rao G

        Issue Links

          Activity

            People

            • Assignee:
              Uma Maheswara Rao G
              Reporter:
              Uma Maheswara Rao G
            • Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development