You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2021/09/17 02:43:37 UTC

[GitHub] [iotdb] cornmonster commented on pull request #3987: [ISSUE-3857] Support show now() function

cornmonster commented on pull request #3987:
URL: https://github.com/apache/iotdb/pull/3987#issuecomment-921411080


   Hi, it is really nice to have a way to check the current server time. 
   
   The relational DBMS(s) like MySQL or PostgreSQL use built-in UDF to handle this need. But I since IoTDB doesn't support syntax like `select now()` (a from clause is required), I can totally understand the way you are doing it.
   
   But since adding a new top-level statement is quite have, I suggest we make most of it and make the SQL API more extendable. Instead of simply shows a system time, maybe we could provide more information like current workload, storage usage, and so on.
   
   In addition, in cluster mode, we should display the information for every single node in the cluster.
   


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org