----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43145/ -----------------------------------------------------------
Review request for hive and John Pullokkaran. Bugs: HIVE-12994 https://issues.apache.org/jira/browse/HIVE-12994 Repository: hive-git Description ------- Implement support for NULLS FIRST/NULLS LAST Diffs ----- itests/src/test/resources/testconfiguration.properties fd2c924380320007245ffd55af5b7809ce4b5201 metastore/if/hive_metastore.thrift 9d8c092ef8ab2d081e014fb84242451e69a37ad8 metastore/scripts/upgrade/derby/033-HIVE-12994.derby.sql PRE-CREATION metastore/scripts/upgrade/derby/hive-schema-2.1.0.derby.sql 660a85a4ab464bd69336d99b231edc47a6657850 metastore/scripts/upgrade/derby/upgrade-2.0.0-to-2.1.0.derby.sql 30de00bf034b5bc4470b4f515cd953586fd93658 metastore/src/gen/protobuf/gen-java/org/apache/hadoop/hive/metastore/Metastore.java 416ae9d68d99110a2d9c2524e4156a43aac18ea6 metastore/src/gen/protobuf/gen-java/org/apache/hadoop/hive/metastore/hbase/HbaseMetastoreProto.java 3b2d7b5db638dc6b35f968279d6b5129977e6f1d metastore/src/gen/thrift/gen-cpp/hive_metastore_types.h c501ac0eb1ed8ced83a557e4843b2c52d7464602 metastore/src/gen/thrift/gen-cpp/hive_metastore_types.cpp 81577b612282f75e09a5a4a3b036fc03b0808ffb metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Order.java cc0e2dd027d2e909ea4a39a742b33218c3f8411f metastore/src/gen/thrift/gen-php/metastore/Types.php 57d1dafcc75f734000f9c8fcd26f54bdcfc2a808 metastore/src/gen/thrift/gen-py/hive_metastore/ttypes.py 77dd9a6fcdaeea31b1d1bc58d3704630000a1d46 metastore/src/gen/thrift/gen-rb/hive_metastore_types.rb 2cf433bff38ca69d8f13350c205e51fd51dab9a1 metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java be54b9ca6e7991e08b95892c8f2a2f460f824434 metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java b8087287446ef243045a017768008829b5c4eb66 metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseUtils.java d6d01bdbb879397d1b28675f5dfaa345d9b1066f metastore/src/model/org/apache/hadoop/hive/metastore/model/MOrder.java 1fa82a4fde9d315d4955ee5698796f58ce8a0921 metastore/src/protobuf/org/apache/hadoop/hive/metastore/hbase/hbase_metastore_proto.proto 466fdf9dc0b0e9d1c58c9b44977896780963e2ac metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseStore.java 29d5a64154fdb9ab9586fb919fbeae7106b1a874 metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseStoreBitVector.java b0d76620b94b25c66b8e08571066660e6a8c8343 metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestSharedStorageDescriptor.java e0d8ce4dede474ad1b752794379342993bf89fa4 ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java be6ea63fca44bf2bfbdc4368adc26c3d1448561d ql/src/java/org/apache/hadoop/hive/ql/exec/MapJoinOperator.java 91b5ca769f88b04c379f38bad3518d36958f1c71 ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java e9cd450276511c36ecbdd0098c083871273f361d ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/HybridHashTableContainer.java fdc1dfffd977ef3a71fcf56da19a3b2b91538921 ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinBytesTableContainer.java 5c2ff9276b575f93a3df0fd2da378b85f7074a22 ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinKey.java cfb9abc380880da2cefe6968fd81dc3b974ec1ed ql/src/java/org/apache/hadoop/hive/ql/exec/vector/reducesink/VectorReduceSinkCommonOperator.java 7bdd11a1e46ac41f16fe26abebe8371fd1e41275 ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 0bab769bb8982782a023363bcb782a5d149854e1 ql/src/java/org/apache/hadoop/hive/ql/optimizer/AbstractSMBJoinProc.java b57dc77c23767f46c27fad0c5761bd2a79489f18 ql/src/java/org/apache/hadoop/hive/ql/optimizer/BucketingSortingReduceSinkOptimizer.java d5f3057a5c070a22786d3151132dc59d38a0ebb4 ql/src/java/org/apache/hadoop/hive/ql/optimizer/ReduceSinkMapJoinProc.java c38c6d7f7b8503fed45ad9034b45064bbe3100dd ql/src/java/org/apache/hadoop/hive/ql/optimizer/SortedDynPartitionOptimizer.java 27b04570b0fc2af41aaa41e83198e640ffc17e80 ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java 02db680c51f7767cd1369bff3b202eaa2367c481 ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/ASTConverter.java 3f2267dc7926243b82d69cc65400b5b536081e53 ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/HiveOpConverter.java a0e374c744c880c6980a8abc7667d36c8ccba7e1 ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java 638b91ec9f6a8cdc7af66d96955c40dca879776c ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkReduceSinkMapJoinProc.java f48fac138033b7cd128c34c3046644c4add838d3 ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java af1ee20a8636523a77b45bf596daddb1330b09d5 ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java e2d404b0bd1d3916e498bc4c4076cdf168ef2272 ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 4c4470be48931b66cc573b8bb0c295999241b182 ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 5f14c6b6851125508b90b5b3886bc71c6c0c6ac3 ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 61bd10c9336a3e294675d055eab85ad9f7ff9122 ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java ba1945f542b68069584182d10433d0a33a847027 ql/src/java/org/apache/hadoop/hive/ql/parse/TypeCheckProcFactory.java 598520c133f56ac515a90c4800dce13b16a0d71e ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java 04d26f310718b7e8066ed1d7e5b3537f04d5101c ql/src/java/org/apache/hadoop/hive/ql/plan/ReduceSinkDesc.java 41d9ffe5ca13863e35d106f41e35a46af0d0ecf1 ql/src/test/org/apache/hadoop/hive/ql/txn/compactor/TestWorker.java cf7eb703a158dd5d1236641eb1f9f004a04d5e7b ql/src/test/queries/clientpositive/order_null.q PRE-CREATION ql/src/test/results/clientpositive/order_null.q.out PRE-CREATION ql/src/test/results/clientpositive/tez/order_null.q.out PRE-CREATION serde/if/serde.thrift 40d5f47391bcb6408ba53f727d3020aadcc29c5c serde/src/gen/thrift/gen-cpp/serde_constants.h 3566ead87ebcf7b76bfb71f0bd9874d7acd5603b serde/src/gen/thrift/gen-cpp/serde_constants.cpp 243d3b8f11a999142c1801bfa7d3c5deab846628 serde/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/serde/serdeConstants.java 8b3eeb763bdab677a520535884abb2893ee85643 serde/src/gen/thrift/gen-php/org/apache/hadoop/hive/serde/Types.php 8370698933b2c2466aa7d5cdf499f640c5920996 serde/src/gen/thrift/gen-py/org_apache_hadoop_hive_serde/constants.py 6ef3bcfd6e7faa448c31722b9ba78d68eb331a6d serde/src/gen/thrift/gen-rb/serde_constants.rb f98441b48b113ddd801755b72b87ed39ad1f4ab2 serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDe.java 144ea5a025e5b0c118ab35230ee4059d08efe38f serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDeWithEndPrefix.java ec43ae3e6480deaa0cc15e96da76dd8e776f3268 serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/fast/BinarySortableSerializeWrite.java 9ea6e914c58effda11a5db93077ed99ab1d75e1f serde/src/test/org/apache/hadoop/hive/serde2/binarysortable/TestBinarySortableFast.java ae476aeeb143b83f1773e40eff0d629150560b76 serde/src/test/org/apache/hadoop/hive/serde2/binarysortable/TestBinarySortableSerDe.java af47e6f7082e464079a62df898d38ceecd9bfb61 Diff: https://reviews.apache.org/r/43145/diff/ Testing ------- Current testsuite to verify that default behavior has not changed. Added q file with different cases. We still need to add additional test cases. Thanks, Jesús Camacho Rodríguez