----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52427/ -----------------------------------------------------------
(Updated Oct. 18, 2016, 3:06 p.m.) Review request for hive, Ashutosh Chauhan and pengcheng xiong. Changes ------- parser-only change ;) Bugs: HIVE-14580 https://issues.apache.org/jira/browse/HIVE-14580 Repository: hive-git Description ------- adds support for || text concatenation * reuses the concat udf for doing the real work * same precedence as + / - ; I think this is expected * small ast rewrite code in SemanticAnalyzer * details about other db || operators: https://issues.apache.org/jira/browse/HIVE-14580?focusedCommentId=15546578&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15546578 Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 5d3fa6aeea5251e461c4b0a8277b4ff1b55f9115 ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 50987c391e435f4e3c71243e90af89fc52f380aa ql/src/test/queries/clientpositive/concat_op.q PRE-CREATION ql/src/test/results/clientpositive/concat_op.q.out PRE-CREATION Diff: https://reviews.apache.org/r/52427/diff/ Testing ------- added qtest Thanks, Zoltan Haindrich