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


Reply via email to