Files
impala/testdata/workloads/functional-query/queries/QueryTest
Paul Rogers d94a474658 IMPALA-7907: Fix ScalarFunction.toSql()
Each AST statement node has a toSql() implementation. The code for
ScalarFunction and ToSqlUtils has a number of issues:

* If Location or Symbol are not set, they are shown as 'null'. Better to
  omit these clauses if the items are not available. This is mostly an
  issue during testing.
* The generated SQL does not follow the CREATE TABLE syntax.  For
  example, the signature and return value are provided for Java
  functions, but should not be.
* Unlike other statements, this one is generated with a trailing newline.
* ToSql.getCreateFunctionSql() fails to separate functions with the
  semi-colon statement separator.

These are all minor issues, but we might as well fix the code to work as
intended.

Testing:
* Added a new unit tests to verify the behavior (no tested existed
  previously.)
* Re-ran all FE tests.

Change-Id: Id34d6df97760a11c299092dff8edbdb7033bce1c
Reviewed-on: http://gerrit.cloudera.org:8080/12014
Reviewed-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>
2018-12-11 02:47:16 +00:00
..
2014-01-08 10:48:09 -08:00
2014-06-20 13:35:10 -07:00