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