PMD Results
The following document contains the results of PMD 5.2.3.
Files
Benchmarks - org/apache/omid/benchmarks/tso/RawTxRunner.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 182–184 |
Benchmarks - org/apache/omid/benchmarks/utils/ZipfianGenerator.java
| Violation | Line |
|---|---|
| Useless parentheses. | 234 |
| Useless parentheses. | 234 |
Commit Table - org/apache/omid/committable/InMemoryCommitTable.java
| Violation | Line |
|---|---|
| Useless parentheses. | 74 |
Common - org/apache/omid/NetworkUtils.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 42–44 |
| Useless parentheses. | 58 |
HBase Client - org/apache/omid/transaction/HBaseAsyncPostCommitter.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 30 |
HBase Client - org/apache/omid/transaction/HBaseSyncPostCommitter.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'metrics'. | 50 |
| Avoid unused method parameters such as 'updateSCFuture'. | 75 |
HBase Client - org/apache/omid/transaction/HBaseTransactionManager.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 49 |
HBase Client - org/apache/omid/transaction/SnapshotFilterImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 94 |
| Avoid unused method parameters such as 'transaction'. | 106 |
| Avoid unused method parameters such as 'commitCache'. | 106 |
| Useless parentheses. | 195 |
| Useless parentheses. | 283 |
| Useless parentheses. | 299 |
| Avoid unused method parameters such as 'attributeMap'. | 321 |
| Useless parentheses. | 423 |
| Useless parentheses. | 472 |
HBase Client - org/apache/omid/transaction/TTable.java
| Violation | Line |
|---|---|
| Useless parentheses. | 134 |
| Useless parentheses. | 150 |
HBase Commit Table - org/apache/omid/committable/hbase/HBaseCommitTable.java
| Violation | Line |
|---|---|
| Useless parentheses. | 109 |
| Useless parentheses. | 142 |
| Useless parentheses. | 274 |
| Useless parentheses. | 278 |
| Useless parentheses. | 282 |
| Useless parentheses. | 332 |
HBase Common - org/apache/omid/committable/hbase/KeyGeneratorImplementations.java
| Violation | Line |
|---|---|
| Useless parentheses. | 56 |
| Useless parentheses. | 91 |
| Useless parentheses. | 146 |
HBase Common - org/apache/omid/transaction/CellInfo.java
| Violation | Line |
|---|---|
| Useless parentheses. | 31 |
| Useless parentheses. | 32 |
HBase Common - org/apache/omid/transaction/CellUtils.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.phoenix.thirdparty.com.google.common.base.Objects' | 41 |
HBase Coprocessors - org/apache/omid/transaction/CompactorScanner.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 137–148 |
| These nested if statements could be combined | 250–252 |
HBase Coprocessors - org/apache/omid/transaction/OmidCompactor.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Queue' | 44 |
| Avoid unused imports such as 'java.util.concurrent.ConcurrentLinkedQueue' | 45 |
| Avoid unused private fields such as 'env'. | 67 |
HBase Coprocessors - org/apache/omid/transaction/OmidSnapshotFilter.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'env'. | 66 |
| Avoid unnecessary return statements | 167 |
HBase Coprocessors - org/apache/omid/transaction/TransactionVisibilityFilterBase.java
| Violation | Line |
|---|---|
| Useless parentheses. | 122 |
| Overriding method merely calls super | 247–249 |
Shims layer for HBase 1.x - org/apache/hadoop/hbase/ipc/controller/InterRegionServerRpcController.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'conf'. | 35 |
State Machine - org/apache/statemachine/StateMachine.java
| Violation | Line |
|---|---|
| Useless parentheses. | 154 |
Transaction Client - org/apache/omid/transaction/AbstractTransaction.java
| Violation | Line |
|---|---|
| Useless parentheses. | 66 |
Transaction Client - org/apache/omid/tso/client/TSOClient.java
| Violation | Line |
|---|---|
| Useless parentheses. | 235 |
TSO and TO Servers - org/apache/omid/tso/AbstractRequestProcessor.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'metrics'. | 62 |
TSO and TO Servers - org/apache/omid/tso/BatchPoolModule.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'config'. | 36 |
TSO and TO Servers - org/apache/omid/tso/CacheEvaluation.java
| Violation | Line |
|---|---|
| Useless parentheses. | 93 |
| Useless parentheses. | 102 |
TSO and TO Servers - org/apache/omid/tso/LowWatermarkWriterImpl.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'metrics'. | 44 |
| Avoid unused constructor parameters such as 'config'. | 47 |
TSO and TO Servers - org/apache/omid/tso/PersistenceProcessorHandler.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 41 |
TSO and TO Servers - org/apache/omid/tso/PersistenceProcessorImpl.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'metrics'. | 63 |
| Avoid unused constructor parameters such as 'commitTable'. | 68 |
TSO and TO Servers - org/apache/omid/tso/RetryProcessorImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 121 |
TSO and TO Servers - org/apache/omid/tso/TSOServer.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'lowWatermarkWriter'. | 57 |
TSO and TO Servers - org/apache/omid/tso/TSOStateManager.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'epoch'. | 36 |
TSO and TO Servers - org/apache/omid/tso/TimestampOracleImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 150 |
| Avoid empty while statements | 151–153 |
| Useless parentheses. | 154 |
| Useless parentheses. | 157 |
| Useless parentheses. | 158 |
TSO and TO Servers - org/apache/omid/tso/WorldClockOracleImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 149 |