hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Zoltan Haindrich <k...@rxd.hu>
Subject Review Request 52427: HIVE-14580 - Introduce || operator
Date Fri, 30 Sep 2016 15:08:20 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52427/
-----------------------------------------------------------

Review request for hive, Ashutosh Chauhan and pengcheng xiong.


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


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g b623187a788a23eceb70a71de62671ad778a24b7

  ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g e6b70a0a0e328a27a808032857b8ecb7ecffeb64

  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 747f3876b76de195fea78d87bfd98bd0ec2b87a0

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


Thanks,

Zoltan Haindrich


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message