Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-18365 Eliminate the findbugs warnings for hbase-common

This commit is contained in:
Chia-Ping Tsai 2017-07-13 19:35:18 +08:00
parent e1bf966f93
commit 1b8fb0a8d8
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hbase-common/src/main/java/org/apache/hadoop/hbase/util/DynamicClassLoader.java
View File

@ -179,8 +179,9 @@ public class DynamicClassLoader extends ClassLoaderBase {
private synchronized void loadNewJars() { private synchronized void loadNewJars() {
// Refresh local jar file lists // Refresh local jar file lists
if (localDir != null) { File[] files = localDir == null ? null : localDir.listFiles();
for (File file : localDir.listFiles()) { if (files != null) {
for (File file : files) {
String fileName = file.getName(); String fileName = file.getName();
if (jarModifiedTime.containsKey(fileName)) { if (jarModifiedTime.containsKey(fileName)) {
continue; continue;

2
hbase-common/src/main/java/org/apache/hadoop/hbase/util/OrderedBytes.java
View File

@ -463,7 +463,7 @@ public class OrderedBytes {
static int lengthVaruint64(PositionedByteRange src, boolean comp) { static int lengthVaruint64(PositionedByteRange src, boolean comp) {
int a0 = (comp ? DESCENDING : ASCENDING).apply(src.peek()) & 0xff; int a0 = (comp ? DESCENDING : ASCENDING).apply(src.peek()) & 0xff;
if (a0 <= 240) return 1; if (a0 <= 240) return 1;
if (a0 >= 241 && a0 <= 248) return 2; if (a0 <= 248) return 2;
if (a0 == 249) return 3; if (a0 == 249) return 3;
if (a0 == 250) return 4; if (a0 == 250) return 4;
if (a0 == 251) return 5; if (a0 == 251) return 5;