----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32189/ -----------------------------------------------------------
(Updated March 20, 2015, 5:24 a.m.) Review request for hive and Jason Dere. Changes ------- I changed bitwise shift operators to <<-, ->>, ->>> because >> conflicts with nested complex type declaration, e.g. ARRAY<MAP<STRING,STRING>> I found that HiveParser.g has the following declaration for array type. : KW_ARRAY LESSTHAN type GREATERTHAN -> ^(TOK_LIST type) I think it uses LESSTHAN and GREATERTHAN in array type declaration to avoid conflicts with < > operators I did not find a way how to do this trick with << >> Bugs: HIVE-9859 https://issues.apache.org/jira/browse/HIVE-9859 Repository: hive-git Description ------- HIVE-9859 Create bitwise left/right shift UDFs Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java fdbfab9e1c4f098766f58e2d07653a44f45d3350 ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g e7de6c86a3c7a674b54f3678b00f34f2dd903dc8 ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g d2d998972b64a19bde28cf176b3f948c00ba492a ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 0a05cebf1f71bd32c8023cdb10c8393a0d871cc2 ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPBitLeftShift.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPBitRightShift.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPBitUnsignedRightShift.java PRE-CREATION ql/src/test/queries/clientpositive/udf_bitwise_left_shift.q PRE-CREATION ql/src/test/queries/clientpositive/udf_bitwise_right_shift.q PRE-CREATION ql/src/test/queries/clientpositive/udf_bitwise_unsigned_right_shift.q PRE-CREATION ql/src/test/results/clientpositive/show_functions.q.out 81abeb9be4fd47724be544c7bc8da8b25fcd6e75 ql/src/test/results/clientpositive/udf_bitwise_left_shift.q.out PRE-CREATION ql/src/test/results/clientpositive/udf_bitwise_right_shift.q.out PRE-CREATION ql/src/test/results/clientpositive/udf_bitwise_unsigned_right_shift.q.out PRE-CREATION Diff: https://reviews.apache.org/r/32189/diff/ Testing ------- Thanks, Alexander Pivovarov