[incubator-iotdb] branch master updated: Change TreeSet to List in getAllDevices (#1756)

[qiaojialin]

This is an automated email from the ASF dual-hosted git repository.

qiaojialin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-iotdb.git


The following commit(s) were added to refs/heads/master by this push:
     new 0bdc1e1  Change TreeSet to List in getAllDevices (#1756)
0bdc1e1 is described below

commit 0bdc1e15cd47a1d64d5a58ea19acf97f86037c8e
Author: Zesong Sun <sz...@mails.tsinghua.edu.cn>
AuthorDate: Thu Sep 24 13:26:23 2020 +0800

    Change TreeSet to List in getAllDevices (#1756)
---
 .../java/org/apache/iotdb/tsfile/read/TsFileSequenceReader.java  | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git 
a/tsfile/src/main/java/org/apache/iotdb/tsfile/read/TsFileSequenceReader.java 
b/tsfile/src/main/java/org/apache/iotdb/tsfile/read/TsFileSequenceReader.java
index ab27a4c..b53d5ff 100644
--- 
a/tsfile/src/main/java/org/apache/iotdb/tsfile/read/TsFileSequenceReader.java
+++ 
b/tsfile/src/main/java/org/apache/iotdb/tsfile/read/TsFileSequenceReader.java
@@ -29,7 +29,6 @@ import java.util.Map;
 import java.util.Map.Entry;
 import java.util.Set;
 import java.util.TreeMap;
-import java.util.TreeSet;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.locks.ReadWriteLock;
 import java.util.concurrent.locks.ReentrantReadWriteLock;
@@ -423,7 +422,7 @@ public class TsFileSequenceReader implements AutoCloseable {
   }
 
   private List<String> getAllDevices(MetadataIndexNode metadataIndexNode) 
throws IOException {
-    Set<String> deviceSet = new TreeSet<>();
+    List<String> deviceList = new ArrayList<>();
     int metadataIndexListSize = metadataIndexNode.getChildren().size();
     for (int i = 0; i < metadataIndexListSize; i++) {
       MetadataIndexEntry metadataIndex = 
metadataIndexNode.getChildren().get(i);
@@ -431,7 +430,7 @@ public class TsFileSequenceReader implements AutoCloseable {
         case LEAF_MEASUREMENT:
         case INTERNAL_MEASUREMENT:
           for (MetadataIndexEntry index : metadataIndexNode.getChildren()) {
-            deviceSet.add(index.getName());
+            deviceList.add(index.getName());
           }
           break;
         case LEAF_DEVICE:
@@ -442,11 +441,11 @@ public class TsFileSequenceReader implements 
AutoCloseable {
           }
           ByteBuffer buffer = readData(metadataIndex.getOffset(), endOffset);
           MetadataIndexNode node = MetadataIndexNode.deserializeFrom(buffer);
-          deviceSet.addAll(getAllDevices(node));
+          deviceList.addAll(getAllDevices(node));
           break;
       }
     }
-    return new ArrayList<>(deviceSet);
+    return deviceList;
   }
 
   /**