shameersss1 commented on a change in pull request #1105: URL: https://github.com/apache/hive/pull/1105#discussion_r452133510
########## File path: parser/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g ########## @@ -734,6 +734,21 @@ dropPartitionOperator EQUAL | NOTEQUAL | LESSTHANOREQUALTO | LESSTHAN | GREATERTHANOREQUALTO | GREATERTHAN ; +filterPartitionSpec + : + LPAREN filterPartitionVal (COMMA filterPartitionVal )* RPAREN -> ^(TOK_PARTSPEC filterPartitionVal +) + ; + +filterPartitionVal + : + identifier filterPartitionOperator constant -> ^(TOK_PARTVAL identifier filterPartitionOperator constant) + ; + +filterPartitionOperator + : + EQUAL | NOTEQUAL | LESSTHANOREQUALTO | LESSTHAN | GREATERTHANOREQUALTO | GREATERTHAN | KW_LIKE Review comment: There is one slight difference apart from the LIKE operator, dropPartitionSpec expects PARTITON keyword, But since in MSCK we already have a PARTITONS keyword in the syntax it will cause a problem. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org