Index: ql/src/java/org/apache/hadoop/hive/ql/udf/UDFAcos.java IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP <+>UTF-8 =================================================================== --- ql/src/java/org/apache/hadoop/hive/ql/udf/UDFAcos.java (revision f53d07b86e98f6a4f85b90a78701b8cad60244af) +++ ql/src/java/org/apache/hadoop/hive/ql/udf/UDFAcos.java (revision fe1afe47e26350ff47424957bd78f06bdfe49533) @@ -44,8 +44,13 @@ */ @Override protected DoubleWritable doEvaluate(DoubleWritable a) { - result.set(Math.acos(a.get())); - return result; + double d = a.get(); + if (d < -1 || d > 1) { + return null; + } else { + result.set(Math.acos(d)); + return result; + } } } Index: ql/src/test/results/clientpositive/udf_acos.q.out IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP <+>US-ASCII =================================================================== --- ql/src/test/results/clientpositive/udf_acos.q.out (revision f53d07b86e98f6a4f85b90a78701b8cad60244af) +++ ql/src/test/results/clientpositive/udf_acos.q.out (revision fe1afe47e26350ff47424957bd78f06bdfe49533) @@ -58,4 +58,4 @@ POSTHOOK: type: QUERY POSTHOOK: Input: default@src #### A masked pattern was here #### -NaN +NULL Index: ql/src/java/org/apache/hadoop/hive/ql/udf/UDFAsin.java IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP <+>UTF-8 =================================================================== --- ql/src/java/org/apache/hadoop/hive/ql/udf/UDFAsin.java (revision fe1afe47e26350ff47424957bd78f06bdfe49533) +++ ql/src/java/org/apache/hadoop/hive/ql/udf/UDFAsin.java (revision 630042704cdadaf69a7afe49ffafb3c4af91a3b8) @@ -44,8 +44,13 @@ */ @Override protected DoubleWritable doEvaluate(DoubleWritable a) { - result.set(Math.asin(a.get())); - return result; + double d = a.get(); + if (d < -1 || d > 1) { + return null; + } else { + result.set(Math.asin(d)); + return result; + } } } Index: ql/src/test/results/clientpositive/udf_asin.q.out IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP <+>US-ASCII =================================================================== --- ql/src/test/results/clientpositive/udf_asin.q.out (revision fe1afe47e26350ff47424957bd78f06bdfe49533) +++ ql/src/test/results/clientpositive/udf_asin.q.out (revision 630042704cdadaf69a7afe49ffafb3c4af91a3b8) @@ -58,4 +58,4 @@ POSTHOOK: type: QUERY POSTHOOK: Input: default@src #### A masked pattern was here #### -NaN +NULL