2009-05-19 04:56:52 +00:00
|
|
|
<FindBugsFilter>
|
|
|
|
<Match>
|
|
|
|
<Package name="org.apache.hadoop.record.compiler.generated" />
|
|
|
|
</Match>
|
2011-06-12 22:00:51 +00:00
|
|
|
<Match>
|
|
|
|
<Package name="org.apache.hadoop.hdfs.protocol.proto" />
|
|
|
|
</Match>
|
2012-04-03 00:59:02 +00:00
|
|
|
<Match>
|
|
|
|
<Package name="org.apache.hadoop.hdfs.server.namenode.ha.proto" />
|
|
|
|
</Match>
|
2012-07-20 00:25:50 +00:00
|
|
|
<Match>
|
|
|
|
<Package name="org.apache.hadoop.hdfs.qjournal.protocol" />
|
|
|
|
</Match>
|
2009-05-19 04:56:52 +00:00
|
|
|
<Match>
|
|
|
|
<Bug pattern="EI_EXPOSE_REP" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Bug pattern="EI_EXPOSE_REP2" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Bug pattern="SE_COMPARATOR_SHOULD_BE_SERIALIZABLE" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Class name="~.*_jsp" />
|
|
|
|
<Bug pattern="DLS_DEAD_LOCAL_STORE" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Field name="_jspx_dependants" />
|
|
|
|
<Bug pattern="UWF_UNWRITTEN_FIELD" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Inconsistent synchronization for Client.Connection.out is
|
|
|
|
is intentional to make a connection to be closed instantly.
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.ipc.Client$Connection" />
|
|
|
|
<Field name="out" />
|
|
|
|
<Bug pattern="IS2_INCONSISTENT_SYNC" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Ignore Cross Scripting Vulnerabilities
|
2011-06-12 22:00:51 +00:00
|
|
|
We have an input quoting filter that protects us.
|
2009-05-19 04:56:52 +00:00
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Bug code="XSS" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Bug code="HRS" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Ignore warnings where child class has the same name as
|
|
|
|
super class. Classes based on Old API shadow names from
|
|
|
|
new API. Should go off after HADOOP-1.0
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="~org.apache.hadoop.mapred.*" />
|
|
|
|
<Bug pattern="NM_SAME_SIMPLE_NAME_AS_SUPERCLASS" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Ignore warnings for usage of System.exit. This is
|
|
|
|
required and have been well thought out
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.Child$2" />
|
|
|
|
<Method name="run" />
|
|
|
|
<Bug pattern="DM_EXIT" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.JobTracker" />
|
|
|
|
<Method name="addHostToNodeMapping" />
|
|
|
|
<Bug pattern="DM_EXIT" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.Task" />
|
|
|
|
<Or>
|
|
|
|
<Method name="done" />
|
|
|
|
<Method name="commit" />
|
|
|
|
<Method name="statusUpdate" />
|
|
|
|
</Or>
|
|
|
|
<Bug pattern="DM_EXIT" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.Task$TaskReporter" />
|
|
|
|
<Method name="run" />
|
|
|
|
<Bug pattern="DM_EXIT" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
We need to cast objects between old and new api objects
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.OutputCommitter" />
|
|
|
|
<Bug pattern="BC_UNCONFIRMED_CAST" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
We intentionally do the get name from the inner class
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.TaskTracker$MapEventsFetcherThread" />
|
|
|
|
<Method name="run" />
|
|
|
|
<Bug pattern="IA_AMBIGUOUS_INVOCATION_OF_INHERITED_OR_OUTER_METHOD" />
|
|
|
|
</Match>
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.FileOutputCommitter" />
|
|
|
|
<Bug pattern="NM_WRONG_PACKAGE_INTENTIONAL" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Ignoring this warning as resolving this would need a non-trivial change in code
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.lib.aggregate.ValueAggregatorBaseDescriptor" />
|
|
|
|
<Method name="configure" />
|
|
|
|
<Field name="maxNumItems" />
|
|
|
|
<Bug pattern="ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Comes from org.apache.jasper.runtime.ResourceInjector. Cannot do much.
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.jobqueue_005fdetails_jsp" />
|
|
|
|
<Field name="_jspx_resourceInjector" />
|
|
|
|
<Bug pattern="SE_BAD_FIELD" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Storing textInputFormat and then passing it as a parameter. Safe to ignore.
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.lib.aggregate.ValueAggregatorJob" />
|
|
|
|
<Method name="createValueAggregatorJob" />
|
|
|
|
<Bug pattern="DLS_DEAD_STORE_OF_CLASS_LITERAL" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
Can remove this after the upgrade to findbugs1.3.8
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.lib.db.DBInputFormat" />
|
|
|
|
<Method name="getSplits" />
|
|
|
|
<Bug pattern="DLS_DEAD_LOCAL_STORE" />
|
|
|
|
</Match>
|
|
|
|
<!--
|
|
|
|
This is a spurious warning. Just ignore
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.mapred.MapTask$MapOutputBuffer" />
|
|
|
|
<Field name="kvindex" />
|
|
|
|
<Bug pattern="IS2_INCONSISTENT_SYNC" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<!--
|
|
|
|
core changes
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="~org.apache.hadoop.*" />
|
|
|
|
<Bug code="MS" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.fs.FileSystem" />
|
|
|
|
<Method name="checkPath" />
|
|
|
|
<Bug pattern="ES_COMPARING_STRINGS_WITH_EQ" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.fs.kfs.KFSOutputStream" />
|
|
|
|
<Field name="path" />
|
|
|
|
<Bug pattern="URF_UNREAD_FIELD" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.fs.kfs.KosmosFileSystem" />
|
|
|
|
<Method name="initialize" />
|
|
|
|
<Bug pattern="DM_EXIT" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.io.Closeable" />
|
|
|
|
<Bug pattern="NM_SAME_SIMPLE_NAME_AS_INTERFACE" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.security.AccessControlException" />
|
|
|
|
<Bug pattern="NM_SAME_SIMPLE_NAME_AS_SUPERCLASS" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.record.meta.Utils" />
|
|
|
|
<Method name="skip" />
|
|
|
|
<Bug pattern="BC_UNCONFIRMED_CAST" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<!--
|
|
|
|
The compareTo method is actually a dummy method that just
|
|
|
|
throws excpetions. So, no need to override equals. Ignore
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.record.meta.RecordTypeInfo" />
|
|
|
|
<Bug pattern="EQ_COMPARETO_USE_OBJECT_EQUALS" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.util.ProcfsBasedProcessTree" />
|
|
|
|
<Bug pattern="DMI_HARDCODED_ABSOLUTE_FILENAME" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<!--
|
|
|
|
Streaming, Examples
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.streaming.StreamUtil$TaskId" />
|
|
|
|
<Bug pattern="URF_UNREAD_FIELD" />
|
|
|
|
</Match>
|
|
|
|
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.examples.DBCountPageView" />
|
|
|
|
<Method name="verify" />
|
|
|
|
<Bug pattern="OBL_UNSATISFIED_OBLIGATION" />
|
|
|
|
</Match>
|
|
|
|
|
2011-06-12 22:00:51 +00:00
|
|
|
<!--
|
|
|
|
getTmpInputStreams is pretty much like a stream constructor.
|
|
|
|
The newly created streams are not supposed to be closed in the constructor. So ignore
|
|
|
|
the OBL warning.
|
|
|
|
-->
|
2009-05-19 04:56:52 +00:00
|
|
|
<Match>
|
2012-04-02 17:38:56 +00:00
|
|
|
<Class name="org.apache.hadoop.hdfs.server.datanode.fsdataset.impl.FsDatasetImpl" />
|
2011-06-12 22:00:51 +00:00
|
|
|
<Method name="getTmpInputStreams" />
|
2009-05-19 04:56:52 +00:00
|
|
|
<Bug pattern="OBL_UNSATISFIED_OBLIGATION" />
|
|
|
|
</Match>
|
|
|
|
|
2009-06-24 05:48:25 +00:00
|
|
|
<!--
|
2011-06-12 22:00:51 +00:00
|
|
|
ResponseProccessor is thread that is designed to catch RuntimeException.
|
2009-06-24 05:48:25 +00:00
|
|
|
-->
|
|
|
|
<Match>
|
2011-06-12 22:00:51 +00:00
|
|
|
<Class name="org.apache.hadoop.hdfs.DFSClient$DFSOutputStream$DataStreamer$ResponseProcessor" />
|
|
|
|
<Method name="run" />
|
|
|
|
<Bug pattern="REC_CATCH_EXCEPTION" />
|
2009-06-24 05:48:25 +00:00
|
|
|
</Match>
|
2011-05-10 23:56:54 +00:00
|
|
|
|
2011-07-29 16:28:45 +00:00
|
|
|
<!--
|
|
|
|
lastAppliedTxid is carefully unsynchronized in the BackupNode in a couple spots.
|
|
|
|
See the comments in BackupImage for justification.
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.namenode.FSImage" />
|
|
|
|
<Field name="lastAppliedTxId" />
|
|
|
|
<Bug pattern="IS2_INCONSISTENT_SYNC" />
|
|
|
|
</Match>
|
2011-06-12 22:00:51 +00:00
|
|
|
<!--
|
|
|
|
Findbugs doesn't realize that closing a FilterOutputStream pushes the close down to
|
|
|
|
wrapped streams, too.
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.namenode.FSImageFormat$Saver" />
|
|
|
|
<Method name="save" />
|
|
|
|
<Bug pattern="OS_OPEN_STREAM" />
|
|
|
|
</Match>
|
2012-03-02 01:37:02 +00:00
|
|
|
<!--
|
|
|
|
the 'metrics' member is sometimes used from synchronized blocks, sometimes not,
|
|
|
|
but it's only reset by test cases, so should be fine
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
|
|
|
|
<Field name="metrics" />
|
|
|
|
<Bug pattern="IS2_INCONSISTENT_SYNC" />
|
|
|
|
</Match>
|
2012-03-02 01:32:49 +00:00
|
|
|
<!--
|
|
|
|
This method isn't performance-critical and is much clearer to write as it's written.
|
|
|
|
-->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.datanode.BlockPoolManager" />
|
|
|
|
<Method name="doRefreshNamenodes" />
|
|
|
|
<Bug category="PERFORMANCE" />
|
|
|
|
</Match>
|
2012-04-09 19:39:58 +00:00
|
|
|
<!-- Don't complain about System.exit() being called from quit() -->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.namenode.MetaRecoveryContext" />
|
|
|
|
<Method name="quit" />
|
|
|
|
<Bug pattern="DM_EXIT" />
|
|
|
|
</Match>
|
2012-07-20 00:25:50 +00:00
|
|
|
|
2012-08-30 21:40:28 +00:00
|
|
|
<!-- Don't complain about recoverBlock equality check -->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.datanode.DataNode" />
|
|
|
|
<Method name="recoverBlock" />
|
|
|
|
<Bug pattern="EC_UNRELATED_TYPES" />
|
|
|
|
</Match>
|
2012-09-05 04:57:47 +00:00
|
|
|
|
2012-07-20 00:25:50 +00:00
|
|
|
<!-- More complex cleanup logic confuses findbugs -->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.qjournal.server.Journal" />
|
|
|
|
<Method name="persistPaxosData" />
|
|
|
|
<Bug pattern="OS_OPEN_STREAM" />
|
|
|
|
</Match>
|
2013-01-08 19:51:44 +00:00
|
|
|
<!-- Don't complain about LocalDatanodeInfo's anonymous class -->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.BlockReaderLocal$LocalDatanodeInfo$1" />
|
|
|
|
<Bug pattern="SE_BAD_FIELD_INNER_CLASS" />
|
|
|
|
</Match>
|
|
|
|
<!-- Only one method increments numFailedVolumes and it is synchronized -->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.datanode.fsdataset.impl.FsVolumeList" />
|
|
|
|
<Field name="numFailedVolumes" />
|
|
|
|
<Bug pattern="VO_VOLATILE_INCREMENT" />
|
|
|
|
</Match>
|
|
|
|
<!-- Access to pendingReceivedRequests is synchronized -->
|
|
|
|
<Match>
|
|
|
|
<Class name="org.apache.hadoop.hdfs.server.datanode.BPServiceActor" />
|
|
|
|
<Method name="notifyNamenodeBlockImmediately" />
|
|
|
|
<Field name="pendingReceivedRequests" />
|
|
|
|
<Bug pattern="VO_VOLATILE_INCREMENT" />
|
|
|
|
</Match>
|
2009-05-19 04:56:52 +00:00
|
|
|
</FindBugsFilter>
|