jcamachor commented on a change in pull request #794: HIVE-21884
URL: https://github.com/apache/hive/pull/794#discussion_r331846838
 
 

 ##########
 File path: metastore/scripts/upgrade/hive/hive-schema-4.0.0.hive.sql
 ##########
 @@ -1552,3 +1611,41 @@ WHERE
   AND (P.`PRINCIPAL_NAME`=current_user() AND P.`PRINCIPAL_TYPE`='USER'
     OR ((array_contains(current_groups(), P.`PRINCIPAL_NAME`) OR 
P.`PRINCIPAL_NAME` = 'public') AND P.`PRINCIPAL_TYPE`='GROUP'))
   AND P.`TBL_PRIV`='SELECT' AND P.`AUTHORIZER`=current_authorizer());
+
+
+drop view SCHEDULED_QUERIES ;
+create or replace view SCHEDULED_QUERIES  as
 
 Review comment:
   We should not add these views to the information schema database which 
should be as close to the standard as possible. I think you did this because 
the `from_unixtime` function cannot be applied when an external table is 
created? However, you can declare the time columns in the external tables with 
`timestamp with local time zone` type. Additionally, that will give the user 
the readable datetime instant.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to