From 8fee2b3ef9a1b4f2cce20dcfeb1b29550b8ecbe9 Mon Sep 17 00:00:00 2001 From: yanhuqing666 Date: Tue, 29 Aug 2017 15:25:21 +0800 Subject: [PATCH] fixed findbugs --- .gitignore | 3 +- findbugs-exclude.xml | 60 +++++++++---------- pom.xml | 2 +- .../nio/handler/MultiNodeQueryHandler.java | 5 +- .../nio/handler/util/TwoTableComparator.java | 12 +--- src/main/java/io/mycat/config/Versions.java | 2 +- .../manager/response/RollbackConfig.java | 2 +- .../mycat/memory/unsafe/array/CharArray.java | 4 +- .../unsafe/memory/mm/MemoryConsumer.java | 2 +- .../unsafe/storage/DataNodeDiskManager.java | 6 +- .../unsafe/storage/DataNodeFileManager.java | 12 ++-- .../memory/unsafe/storage/DiskRowWriter.java | 13 +--- .../unsafe/utils/sort/RowPrefixComputer.java | 1 - .../utils/sort/UnsafeExternalRowSorter.java | 1 - .../utils/sort/UnsafeSorterSpillWriter.java | 2 +- .../mycat/sqlengine/mpp/tmp/RowDataCmp.java | 1 - .../unsafe/storage/BlockManagerTest.java | 2 +- 17 files changed, 51 insertions(+), 79 deletions(-) diff --git a/.gitignore b/.gitignore index 1e6dbfc84..b6d60d5af 100644 --- a/.gitignore +++ b/.gitignore @@ -121,4 +121,5 @@ src/main/resources/zkconf/rule.xml conf/dnindex.properties version.txt copyResources.bat -dependency-reduced-pom.xml \ No newline at end of file +dependency-reduced-pom.xml +checkstyle-result.out \ No newline at end of file diff --git a/findbugs-exclude.xml b/findbugs-exclude.xml index fba61d437..0ce981bce 100644 --- a/findbugs-exclude.xml +++ b/findbugs-exclude.xml @@ -30,10 +30,6 @@ - - - - @@ -84,30 +80,6 @@ - - - - - - - - - - - - - - - - - - - - - - - - @@ -156,7 +128,7 @@ - + @@ -173,11 +145,36 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -187,13 +184,14 @@ + - + diff --git a/pom.xml b/pom.xml index 764143137..44a2b6542 100644 --- a/pom.xml +++ b/pom.xml @@ -17,7 +17,7 @@ yyyy-MM-dd HH:mm:ss version.txt.template version.txt - MyCat Server (OpenCloundDB) + MyCat Server (OpenCloudDB) UTF-8 diff --git a/src/main/java/io/mycat/backend/mysql/nio/handler/MultiNodeQueryHandler.java b/src/main/java/io/mycat/backend/mysql/nio/handler/MultiNodeQueryHandler.java index 1ca462eca..b26ec4503 100644 --- a/src/main/java/io/mycat/backend/mysql/nio/handler/MultiNodeQueryHandler.java +++ b/src/main/java/io/mycat/backend/mysql/nio/handler/MultiNodeQueryHandler.java @@ -397,6 +397,7 @@ public class MultiNodeQueryHandler extends MultiNodeHandler implements LoadDataR if (rrs != null && rrs.getStatement() != null) { netInBytes += rrs.getStatement().getBytes().length; } + assert rrs != null; QueryResult queryResult = new QueryResult(session.getSource().getUser(), rrs.getSqlType(), rrs.getStatement(), selectRows, netInBytes, netOutBytes, startTime, System.currentTimeMillis(), resultSize); QueryResultDispatcher.dispatchQuery(queryResult); @@ -658,9 +659,7 @@ public class MultiNodeQueryHandler extends MultiNodeHandler implements LoadDataR eof[3] = ++packetId; buffer = source.writeToBuffer(eof, buffer); source.write(buffer); - if (dataMergeSvr != null) { - dataMergeSvr.onRowMetaData(columToIndx, fieldCount); - } + dataMergeSvr.onRowMetaData(columToIndx, fieldCount); } public void handleDataProcessException(Exception e) { diff --git a/src/main/java/io/mycat/backend/mysql/nio/handler/util/TwoTableComparator.java b/src/main/java/io/mycat/backend/mysql/nio/handler/util/TwoTableComparator.java index ce3658c6c..cc7045ce5 100644 --- a/src/main/java/io/mycat/backend/mysql/nio/handler/util/TwoTableComparator.java +++ b/src/main/java/io/mycat/backend/mysql/nio/handler/util/TwoTableComparator.java @@ -30,26 +30,20 @@ public class TwoTableComparator implements Comparator { public TwoTableComparator(List fps1, List fps2, List leftOrders, List rightOrders, boolean isAllPushDown, HandlerType type, String charset) { - boolean isAllPushDown1 = isAllPushDown; - HandlerType type1 = type; this.leftFields = HandlerTool.createFields(fps1); this.rightFields = HandlerTool.createFields(fps2); ascs = new ArrayList<>(); for (Order order : leftOrders) { ascs.add(order.getSortOrder() == SQLOrderingSpecification.ASC); } - List leftCmpItems = new ArrayList<>(); - List rightCmpItems = new ArrayList<>(); cmptors = new ArrayList<>(); for (int index = 0; index < ascs.size(); index++) { Order leftOrder = leftOrders.get(index); Order rightOrder = rightOrders.get(index); - Item leftCmpItem = HandlerTool.createItem(leftOrder.getItem(), leftFields, 0, isAllPushDown1, type1, + Item leftCmpItem = HandlerTool.createItem(leftOrder.getItem(), leftFields, 0, isAllPushDown, type, charset); - leftCmpItems.add(leftCmpItem); - Item rightCmpItem = HandlerTool.createItem(rightOrder.getItem(), rightFields, 0, isAllPushDown1, - type1, charset); - rightCmpItems.add(rightCmpItem); + Item rightCmpItem = HandlerTool.createItem(rightOrder.getItem(), rightFields, 0, isAllPushDown, + type, charset); ArgComparator cmptor = new ArgComparator(leftCmpItem, rightCmpItem); cmptor.setCmpFunc(null, leftCmpItem, rightCmpItem, false); cmptors.add(cmptor); diff --git a/src/main/java/io/mycat/config/Versions.java b/src/main/java/io/mycat/config/Versions.java index 50d04cd81..646c24705 100644 --- a/src/main/java/io/mycat/config/Versions.java +++ b/src/main/java/io/mycat/config/Versions.java @@ -31,7 +31,7 @@ public abstract class Versions { public static final byte PROTOCOL_VERSION = 10; private static byte[] serverVersion = "5.6.29-mycat-2.17.08.0-dev-20170824134330".getBytes(); - public static final byte[] VERSION_COMMENT = "MyCat Server (OpenCloundDB)".getBytes(); + public static final byte[] VERSION_COMMENT = "MyCat Server (OpenCloudDB)".getBytes(); public static final String ANNOTATION_NAME = "mycat:"; public static final String ROOT_PREFIX = "mycat"; diff --git a/src/main/java/io/mycat/manager/response/RollbackConfig.java b/src/main/java/io/mycat/manager/response/RollbackConfig.java index bdc24ea8c..235534bbc 100644 --- a/src/main/java/io/mycat/manager/response/RollbackConfig.java +++ b/src/main/java/io/mycat/manager/response/RollbackConfig.java @@ -157,7 +157,7 @@ public final class RollbackConfig { } } // INIT FAILED - if (!rollbackStatus && dataHosts != null) { + if (!rollbackStatus) { for (PhysicalDBPool dn : dataHosts.values()) { dn.clearDataSources("rollbackup config"); dn.stopHeartbeat(); diff --git a/src/main/java/io/mycat/memory/unsafe/array/CharArray.java b/src/main/java/io/mycat/memory/unsafe/array/CharArray.java index d6306033a..cff231013 100644 --- a/src/main/java/io/mycat/memory/unsafe/array/CharArray.java +++ b/src/main/java/io/mycat/memory/unsafe/array/CharArray.java @@ -2,7 +2,6 @@ package io.mycat.memory.unsafe.array; import io.mycat.memory.unsafe.Platform; import io.mycat.memory.unsafe.memory.MemoryBlock; -import io.mycat.memory.unsafe.memory.mm.MemoryConsumer; /** * @author Hash Zhang @@ -18,13 +17,12 @@ public class CharArray { private final long length; - public CharArray(MemoryBlock memory, MemoryConsumer memoryConsumer) { + public CharArray(MemoryBlock memory) { assert memory.size() < (long) Integer.MAX_VALUE * 2 : "Array size > 4 billion elements"; this.memory = memory; this.baseObj = memory.getBaseObject(); this.baseOffset = memory.getBaseOffset(); this.length = memory.size() / WIDTH; - MemoryConsumer memoryConsumer1 = memoryConsumer; } diff --git a/src/main/java/io/mycat/memory/unsafe/memory/mm/MemoryConsumer.java b/src/main/java/io/mycat/memory/unsafe/memory/mm/MemoryConsumer.java index f24c2a3a1..6fffbac8e 100644 --- a/src/main/java/io/mycat/memory/unsafe/memory/mm/MemoryConsumer.java +++ b/src/main/java/io/mycat/memory/unsafe/memory/mm/MemoryConsumer.java @@ -116,7 +116,7 @@ public abstract class MemoryConsumer { throw new OutOfMemoryError("Unable to acquire " + required + " bytes of memory, got " + got); } used += required; - return new CharArray(page, this); + return new CharArray(page); } /** diff --git a/src/main/java/io/mycat/memory/unsafe/storage/DataNodeDiskManager.java b/src/main/java/io/mycat/memory/unsafe/storage/DataNodeDiskManager.java index dd29db634..94bd98c5a 100644 --- a/src/main/java/io/mycat/memory/unsafe/storage/DataNodeDiskManager.java +++ b/src/main/java/io/mycat/memory/unsafe/storage/DataNodeDiskManager.java @@ -4,7 +4,6 @@ package io.mycat.memory.unsafe.storage; import io.mycat.memory.unsafe.utils.MycatPropertyConf; import java.io.File; -import java.io.FileOutputStream; import java.io.IOException; /** @@ -31,12 +30,11 @@ public class DataNodeDiskManager { * cases. */ public DiskRowWriter getDiskWriter( - ConnectionId blockId, File file, SerializerInstance serializerInstance, int bufferSize) throws IOException { boolean syncWrites = conf.getBoolean("server.merge.sync", false); - return new DiskRowWriter(file, serializerInstance, bufferSize, new FileOutputStream(file), - syncWrites, blockId); + return new DiskRowWriter(file, serializerInstance, bufferSize, + syncWrites); } } diff --git a/src/main/java/io/mycat/memory/unsafe/storage/DataNodeFileManager.java b/src/main/java/io/mycat/memory/unsafe/storage/DataNodeFileManager.java index 844b9b11b..ba2871d0a 100644 --- a/src/main/java/io/mycat/memory/unsafe/storage/DataNodeFileManager.java +++ b/src/main/java/io/mycat/memory/unsafe/storage/DataNodeFileManager.java @@ -18,6 +18,11 @@ package io.mycat.memory.unsafe.storage; +import io.mycat.memory.unsafe.utils.JavaUtils; +import io.mycat.memory.unsafe.utils.MycatPropertyConf; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; + import java.io.File; import java.io.IOException; import java.util.ArrayList; @@ -26,12 +31,6 @@ import java.util.UUID; import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentMap; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import io.mycat.memory.unsafe.utils.JavaUtils; -import io.mycat.memory.unsafe.utils.MycatPropertyConf; - /** * Creates and maintains the logical mapping between logical blocks and physical on-disk @@ -60,7 +59,6 @@ public class DataNodeFileManager { public DataNodeFileManager(MycatPropertyConf conf, boolean deleteFilesOnStop) throws IOException { - MycatPropertyConf conf1 = conf; this.deleteFilesOnStop = deleteFilesOnStop; diff --git a/src/main/java/io/mycat/memory/unsafe/storage/DiskRowWriter.java b/src/main/java/io/mycat/memory/unsafe/storage/DiskRowWriter.java index 3fc006f0c..14e399eca 100644 --- a/src/main/java/io/mycat/memory/unsafe/storage/DiskRowWriter.java +++ b/src/main/java/io/mycat/memory/unsafe/storage/DiskRowWriter.java @@ -80,19 +80,15 @@ public class DiskRowWriter extends OutputStream { File file, SerializerInstance serializerInstance, int bufferSize, - OutputStream compressStream, - boolean syncWrites, - ConnectionId blockId) throws IOException { + boolean syncWrites) throws IOException { this.file = file; this.serializerInstance = serializerInstance; this.bufferSize = bufferSize; - OutputStream compressStream1 = compressStream; this.syncWrites = syncWrites; /* ShuffleWriteMetrics writeMetrics, */ - ConnectionId blockId1 = blockId; initialPosition = file.length(); reportedPosition = initialPosition; } @@ -148,20 +144,13 @@ public class DiskRowWriter extends OutputStream { * Flush the partial writes and commit them as a single atomic block. */ public void commitAndClose() throws IOException { - long finalPosition = -1; if (initialized) { // NOTE: Because Kryo doesn’t flush the underlying stream we explicitly flush both the // serializer stream and the lower level stream. objOut.flush(); bs.flush(); close(); - finalPosition = file.length(); - // In certain compression codecs, more bytes are written after close() is called - //writeMetrics.incBytesWritten(finalPosition - reportedPosition) - } else { - finalPosition = file.length(); } - boolean commitAndCloseHasBeenCalled = true; } diff --git a/src/main/java/io/mycat/memory/unsafe/utils/sort/RowPrefixComputer.java b/src/main/java/io/mycat/memory/unsafe/utils/sort/RowPrefixComputer.java index 655c977b5..f8cf2c8c2 100644 --- a/src/main/java/io/mycat/memory/unsafe/utils/sort/RowPrefixComputer.java +++ b/src/main/java/io/mycat/memory/unsafe/utils/sort/RowPrefixComputer.java @@ -16,7 +16,6 @@ public class RowPrefixComputer extends UnsafeExternalRowSorter.PrefixComputer { private final ColMeta colMeta; public RowPrefixComputer(StructType schema) { - StructType schema1 = schema; /** * get the index of the first key word of order */ diff --git a/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeExternalRowSorter.java b/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeExternalRowSorter.java index a996ea519..1961d683e 100644 --- a/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeExternalRowSorter.java +++ b/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeExternalRowSorter.java @@ -262,7 +262,6 @@ public final class UnsafeExternalRowSorter { row1.pointTo(baseObj1, baseOff1, -1); row2.pointTo(baseObj2, baseOff2, -1); int cmp = 0; - int len = orderCols.length; int type = OrderCol.COL_ORDER_TYPE_ASC; diff --git a/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeSorterSpillWriter.java b/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeSorterSpillWriter.java index bdd637537..6ac625486 100644 --- a/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeSorterSpillWriter.java +++ b/src/main/java/io/mycat/memory/unsafe/utils/sort/UnsafeSorterSpillWriter.java @@ -63,7 +63,7 @@ public final class UnsafeSorterSpillWriter { // Our write path doesn't actually use this serializer (since we end up calling the `write()` // OutputStream methods), but DiskRowWriter still calls some methods on it. To work // around this, we pass a dummy no-op serializer. - writer = blockManager.getDiskWriter(conId, file, DummySerializerInstance.INSTANCE, fileBufferSize/**,writeMetrics*/); + writer = blockManager.getDiskWriter(file, DummySerializerInstance.INSTANCE, fileBufferSize/**,writeMetrics*/); // Write the number of records writeIntToBuffer(numRecordsToWrite, 0); writer.write(writeBuffer, 0, 4); diff --git a/src/main/java/io/mycat/sqlengine/mpp/tmp/RowDataCmp.java b/src/main/java/io/mycat/sqlengine/mpp/tmp/RowDataCmp.java index 7e50ec4d7..bd46bc8a7 100644 --- a/src/main/java/io/mycat/sqlengine/mpp/tmp/RowDataCmp.java +++ b/src/main/java/io/mycat/sqlengine/mpp/tmp/RowDataCmp.java @@ -21,7 +21,6 @@ public class RowDataCmp implements Comparator { public int compare(RowDataPacket o1, RowDataPacket o2) { OrderCol[] tmp = this.orderCols; int cmp = 0; - int len = tmp.length; //compare the columns of order by int type = OrderCol.COL_ORDER_TYPE_ASC; for (OrderCol aTmp : tmp) { diff --git a/src/test/java/io/mycat/memory/unsafe/storage/BlockManagerTest.java b/src/test/java/io/mycat/memory/unsafe/storage/BlockManagerTest.java index 336a1a30c..1e7f26339 100644 --- a/src/test/java/io/mycat/memory/unsafe/storage/BlockManagerTest.java +++ b/src/test/java/io/mycat/memory/unsafe/storage/BlockManagerTest.java @@ -61,7 +61,7 @@ public class BlockManagerTest { File file2 = diskBlockManager.getFile("mycat1"); DiskRowWriter writer = blockManager. - getDiskWriter(null, file2, DummySerializerInstance.INSTANCE, 1024 * 1024); + getDiskWriter(file2, DummySerializerInstance.INSTANCE, 1024 * 1024); byte[] writeBuffer = new byte[4]; int v = 4; writeBuffer[0] = (byte) (v >>> 24);