SteveYurongSu commented on a change in pull request #1367:
URL: https://github.com/apache/iotdb/pull/1367#discussion_r509012049



##########
File path: server/src/main/java/org/apache/iotdb/db/conf/IoTDBConfig.java
##########
@@ -131,10 +131,17 @@
    */
   private long allocateMemoryForRead = Runtime.getRuntime().maxMemory() * 3 / 
10;
 
+  /**
+   * Memory allocated for the read process besides cache
+   */
+  private long allocateMemoryForReadWithoutCache = 
Runtime.getRuntime().maxMemory() * 9 / 100;
+
+  private volatile int maxQueryDeduplicatedPathNum = 1000;
+
   /**
    * Is dynamic parameter adapter enable.
    */
-  private boolean enableParameterAdapter = true;
+  private boolean enableParameterAdapter = false;

Review comment:
       Why set it to false? 

##########
File path: server/src/main/java/org/apache/iotdb/db/metadata/MTree.java
##########
@@ -777,18 +777,26 @@ boolean checkStorageGroupByPath(PartialPath path) {
    *
    * @param prefixPath a prefix path or a full path, may contain '*'.
    */
-  List<PartialPath> getAllTimeseriesPathWithAlias(PartialPath prefixPath) 
throws MetadataException {
+  Pair<List<PartialPath>, Integer> getAllTimeseriesPathWithAlias(PartialPath 
prefixPath, int limit, int offset) throws MetadataException {

Review comment:
       Please update the Javadoc...




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to