neuyilan commented on a change in pull request #460: URL: https://github.com/apache/iotdb/pull/460#discussion_r537375072
########## File path: server/src/main/java/org/apache/iotdb/db/metadata/PartialPath.java ########## @@ -20,24 +20,29 @@ import java.util.ArrayList; import java.util.Arrays; +import java.util.Collections; import java.util.List; import org.apache.iotdb.db.conf.IoTDBConstant; import org.apache.iotdb.db.exception.metadata.IllegalPathException; import org.apache.iotdb.db.utils.TestOnly; import org.apache.iotdb.tsfile.common.constant.TsFileConstant; import org.apache.iotdb.tsfile.read.common.Path; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; /** * A prefix path, suffix path or fullPath generated from SQL. * Usually used in the IoTDB server module */ public class PartialPath extends Path implements Comparable<Path> { + private static final Logger logger = LoggerFactory.getLogger(PartialPath.class); + private String[] nodes; // alias of measurement - private String measurementAlias = null; + private String measurementAlias = ""; // alias of time series used in SELECT AS - private String tsAlias = null; + private String tsAlias = ""; Review comment: as far as I know, iotdb does not support empty string as an alias, so it's ok to use an empty string here. ---------------------------------------------------------------- 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: us...@infra.apache.org