pan3793 commented on a change in pull request #1391:
URL: https://github.com/apache/incubator-kyuubi/pull/1391#discussion_r749358007
##########
File path:
externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/operation/GetFunctions.scala
##########
@@ -46,12 +46,20 @@ class GetFunctions(
new StructType()
.add(FUNCTION_CAT, "string", nullable = true, "Function catalog (may be
null)")
.add(FUNCTION_SCHEM, "string", nullable = true, "Function schema (may be
null)")
- .add(FUNCTION_NAME, "string", nullable = true, "Function name. This is
the name used to" +
- " invoke the function")
+ .add(
+ FUNCTION_NAME,
+ "string",
+ nullable = true,
+ "Function name. This is the name used to" +
+ " invoke the function")
.add(REMARKS, "string", nullable = true, "Explanatory comment on the
function")
.add(FUNCTION_TYPE, "int", nullable = true, "Kind of function.")
- .add(SPECIFIC_NAME, "string", nullable = true, "The name which uniquely
identifies this" +
- " function within its schema")
+ .add(
+ SPECIFIC_NAME,
+ "string",
+ nullable = true,
+ "The name which uniquely identifies this" +
+ " function within its schema")
Review comment:
ditto
##########
File path:
externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/operation/GetFunctions.scala
##########
@@ -46,12 +46,20 @@ class GetFunctions(
new StructType()
.add(FUNCTION_CAT, "string", nullable = true, "Function catalog (may be
null)")
.add(FUNCTION_SCHEM, "string", nullable = true, "Function schema (may be
null)")
- .add(FUNCTION_NAME, "string", nullable = true, "Function name. This is
the name used to" +
- " invoke the function")
+ .add(
+ FUNCTION_NAME,
+ "string",
+ nullable = true,
+ "Function name. This is the name used to" +
+ " invoke the function")
Review comment:
ditto
--
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]
