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

Review request for hive and Alexander Pivovarov.


Bugs: HIVE-10788
    https://issues.apache.org/jira/browse/HIVE-10788


Repository: hive-git


Description
-------

Currently sort_array only support primitive types. As we already support 
comparison between non-primitive types, it makes sense to remove this 
restriction.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFSortArray.java 
2d6d58c 
  
ql/src/test/org/apache/hadoop/hive/ql/udf/generic/TestGenericUDFSortArray.java 
PRE-CREATION 
  ql/src/test/queries/clientnegative/udf_sort_array_wrong3.q 034de06 
  ql/src/test/queries/clientpositive/udf_sort_array.q 313bcf8 
  ql/src/test/results/clientnegative/udf_sort_array_wrong1.q.out 075f49b 
  ql/src/test/results/clientnegative/udf_sort_array_wrong2.q.out c068ecd 
  ql/src/test/results/clientnegative/udf_sort_array_wrong3.q.out abf7124 
  ql/src/test/results/clientpositive/udf_sort_array.q.out 9631c2d 

Diff: https://reviews.apache.org/r/34798/diff/


Testing
-------

I added a few tests to udf_sort_array.q, and also added unit tests.


Thanks,

Chao Sun

Reply via email to