----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14280/ -----------------------------------------------------------
Review request for drill. Repository: drill-git Description ------- Implemented trigonometric functions to math functions collection as part of DRILL-219. Implemented functions: COS, SIN, TAN, ACOS, ASIN, ATAN, COSH, SINH, TANH Diffs ----- common/src/main/java/org/apache/drill/common/expression/fn/MathFunctions.java ee3a099 exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/MathFunctions.java 288760b exec/ref/src/main/java/org/apache/drill/exec/ref/eval/fn/MathEvaluators.java 347a4f3 exec/ref/src/test/java/org/apache/drill/exec/ref/RunSimplePlan.java f4992e0 exec/ref/src/test/resources/simple_math.json PRE-CREATION Diff: https://reviews.apache.org/r/14280/diff/ Testing ------- Yes. mvn test -Dtest=RunSimplePlan#mathPlan Output: { "tanh" : 1.0, "cosh" : 1.4524424832623713E13, "atan" : 1.5385494443596428, "asin" : "NaN", "cos" : 0.9147423578045313, "sin" : -0.404037645323065, "sinh" : 1.4524424832623713E13, "acos" : "NaN", "tan" : -0.441695568020698 } { "tanh" : 1.0, "cosh" : 1.0732178989295803E14, "atan" : 1.5405025668761214, "asin" : "NaN", "cos" : -0.013276747223059479, "sin" : 0.9999118601072672, "sinh" : 1.0732178989295803E14, "acos" : "NaN", "tan" : -75.31301480008509 } { "tanh" : 1.0, "cosh" : 1.0732178989295803E14, "atan" : 1.5405025668761214, "asin" : "NaN", "cos" : -0.013276747223059479, "sin" : 0.9999118601072672, "sinh" : 1.0732178989295803E14, "acos" : "NaN", "tan" : -75.31301480008509 } { "tanh" : 1.0, "cosh" : 2.9173087126372744E14, "atan" : 1.5413930385908916, "asin" : "NaN", "cos" : -0.8485702747846052, "sin" : 0.5290826861200238, "sinh" : 2.9173087126372744E14, "acos" : "NaN", "tan" : -0.6234989627162255 } { "tanh" : 1.0, "cosh" : 2.9173087126372744E14, "atan" : 1.5413930385908916, "asin" : "NaN", "cos" : -0.8485702747846052, "sin" : 0.5290826861200238, "sinh" : 2.9173087126372744E14, "acos" : "NaN", "tan" : -0.6234989627162255 } Thanks, Yash Sharma
