Uploaded image for project: '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
    • Status: Closed
    • Priority: 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.

        Attachments

        1. 5598.part2.patch
          24 kB
          Nicolas Liochon
        2. 5598.part2.patch
          24 kB
          Nicolas Liochon
        3. 5598.part1.patch
          23 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:
                umamaheswararao Uma Maheswara Rao G
                Reporter:
                umamaheswararao Uma Maheswara Rao G
              • Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: