kr11 commented on pull request #2024: URL: https://github.com/apache/iotdb/pull/2024#issuecomment-727570771
> Just a little suggestion :For the sql statement, I think if our sql supports both lowercase and uppercase , maybe the value of the index type should be consistent with it . That is to say , If we can also support "INDEX=rtree_paa" , will it be better? > Of course, keep the status quo is also good :) Yes, when we parse the query plan, we first uppercase the input "rtree_paa", which can be seen at : `org.apache.iotdb.db.qp.strategy.LogicalGenerator#enterIndexWithClause` `org.apache.iotdb.db.index.common.IndexType#getIndexType` ---------------------------------------------------------------- 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