comphead commented on code in PR #6037: URL: https://github.com/apache/arrow-datafusion/pull/6037#discussion_r1170500753
########## datafusion/core/tests/sqllogictests/test_files/math.slt: ########## @@ -87,3 +87,9 @@ query RRRRRRRR SELECT round(125.2345, -3), round(125.2345, -2), round(125.2345, -1), round(125.2345), round(125.2345, 0), round(125.2345, 1), round(125.2345, 2), round(125.2345, 3) ---- 0 100 130 125 125 125.2 125.23 125.235 + +# atan2 +query RRRR Review Comment: please include also tests with nulls. Its likely gap from prev implementation `atan2(null, 1.0), atan2(2.0, null)` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
