kgyrtkirk commented on a change in pull request #794: HIVE-21884 URL: https://github.com/apache/hive/pull/794#discussion_r335332639
########## File path: ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g ########## @@ -2066,6 +2074,71 @@ dropMaterializedViewStatement : KW_DROP KW_MATERIALIZED KW_VIEW ifExists? viewName -> ^(TOK_DROP_MATERIALIZED_VIEW viewName ifExists?) ; +createScheduledQueryStatement +@init { pushMsg("create scheduled query statement", state); } +@after { popMsg(state); } + : KW_CREATE KW_SCHEDULED KW_QUERY name=identifier + scheduleSpec + executedAsSpec? Review comment: the thing is that to have the base feature work; we already need impersonation to work - I already have tests for that...so not adding it to the language doesn't really remove any value... only admins may create/alter scheduled queries with executed as option ---------------------------------------------------------------- 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org