----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35968/#review89671 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/udf/UDFCosineSimilarity.java (line 32) <https://reviews.apache.org/r/35968/#comment142333> You're right, we could potentially extend this method. ql/src/java/org/apache/hadoop/hive/ql/udf/UDFCosineSimilarity.java (line 72) <https://reviews.apache.org/r/35968/#comment142335> These still throw HiveException, but I've included all of the code in evaluate() in try/catch, and catch returns null. ql/src/java/org/apache/hadoop/hive/ql/udf/UDFCosineSimilarity.java (line 88) <https://reviews.apache.org/r/35968/#comment142336> These still throw HiveException, but I've included all of the code in evaluate() in try/catch, and catch returns null. ql/src/java/org/apache/hadoop/hive/ql/udf/UDFCosineSimilarity.java (line 109) <https://reviews.apache.org/r/35968/#comment142337> These still throw HiveException, but I've included all of the code in evaluate() in try/catch, and catch returns null. ql/src/java/org/apache/hadoop/hive/ql/udf/UDFCosineSimilarity.java (line 122) <https://reviews.apache.org/r/35968/#comment142338> These still throw HiveException, but I've included all of the code in evaluate() in try/catch, and catch returns null. - Nishant Kelkar On June 28, 2015, 11:22 a.m., Nishant Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35968/ > ----------------------------------------------------------- > > (Updated June 28, 2015, 11:22 a.m.) > > > Review request for hive and Alexander Pivovarov. > > > Repository: hive-git > > > Description > ------- > > 1. Added preliminary UDF code for cosine similarity. 2. Added unit tests and > integration tests. 3. Registered the UDF in the FunctionRegistry class. > > > Diffs > ----- > > .reviewboardrc abc33f91a44b76573cbba334c33417307c63956f > ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java > fabc21e2092561cbf98c35a406e4ee40e71fe1de > ql/src/java/org/apache/hadoop/hive/ql/udf/UDFCosineSimilarity.java > PRE-CREATION > ql/src/test/org/apache/hadoop/hive/ql/udf/TestUDFCosineSimilarity.java > PRE-CREATION > ql/src/test/queries/clientnegative/udf_cosine_similarity_error_1.q > PRE-CREATION > ql/src/test/queries/clientnegative/udf_cosine_similarity_wrongargs_1.q > PRE-CREATION > ql/src/test/queries/clientnegative/udf_cosine_similarity_wrongargs_2.q > PRE-CREATION > ql/src/test/queries/clientnegative/udf_cosine_similarity_wrongargs_3.q > PRE-CREATION > ql/src/test/queries/clientpositive/udf_cosine_similarity.q PRE-CREATION > ql/src/test/results/clientnegative/udf_cosine_similarity_error_1.q.out > PRE-CREATION > ql/src/test/results/clientnegative/udf_cosine_similarity_error_2.q.out > PRE-CREATION > ql/src/test/results/clientnegative/udf_cosine_similarity_wrongargs_1.q.out > PRE-CREATION > ql/src/test/results/clientnegative/udf_cosine_similarity_wrongargs_2.q.out > PRE-CREATION > ql/src/test/results/clientnegative/udf_cosine_similarity_wrongargs_3.q.out > PRE-CREATION > ql/src/test/results/clientpositive/show_functions.q.out > 5de4ffcd1ace477af026b83fb7bfb8068fc192b3 > ql/src/test/results/clientpositive/udf_cosine_similarity.q.out PRE-CREATION > > Diff: https://reviews.apache.org/r/35968/diff/ > > > Testing > ------- > > > Thanks, > > Nishant Kelkar > >