okumin commented on code in PR #5394:
URL: https://github.com/apache/hive/pull/5394#discussion_r1734530696


##########
parser/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g:
##########
@@ -2438,7 +2438,6 @@ primitiveType
     | KW_DATETIME      ->    TOK_DATETIME
     | KW_TIMESTAMP     ->    TOK_TIMESTAMP
     | KW_TIMESTAMPLOCALTZ   ->    TOK_TIMESTAMPLOCALTZ
-    //| KW_TIMESTAMPTZ   ->    TOK_TIMESTAMPTZ
     | KW_TIMESTAMP KW_WITH KW_LOCAL KW_TIME KW_ZONE -> TOK_TIMESTAMPLOCALTZ
     //| KW_TIMESTAMP KW_WITH KW_TIME KW_ZONE -> TOK_TIMESTAMPTZ

Review Comment:
   You'll be right. I removed it.
   
https://github.com/apache/hive/pull/5394/commits/03207b02c11b914cabf17d993621c0fdbb7802f2
   
   Also, I found some more extra words while addressing HIVE-19741. I cleaned 
them up in the same way.
   
https://github.com/apache/hive/pull/5394/commits/a808727e2c6444b5089c595690dfcaa30269ebda



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to