Skip to content

Commit 1117fc3

Browse files
wangyumdongjoon-hyun
authored andcommitted
[SPARK-25760][SQL] Set AddJarCommand return empty
## What changes were proposed in this pull request? Only `AddJarCommand` return `0`, the user will be confused about what it means. This PR sets it to empty. ```sql spark-sql> add jar /Users/yumwang/spark/sql/hive/src/test/resources/TestUDTF.jar; ADD JAR /Users/yumwang/spark/sql/hive/src/test/resources/TestUDTF.jar 0 spark-sql> ``` ## How was this patch tested? manual tests ```sql spark-sql> add jar /Users/yumwang/spark/sql/hive/src/test/resources/TestUDTF.jar; ADD JAR /Users/yumwang/spark/sql/hive/src/test/resources/TestUDTF.jar spark-sql> ``` Closes #22747 from wangyum/AddJarCommand. Authored-by: Yuming Wang <yumwang@ebay.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent 734c6af commit 1117fc3

File tree

1 file changed

+1
-1
lines changed
  • sql/core/src/main/scala/org/apache/spark/sql/execution/command

1 file changed

+1
-1
lines changed

sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ case class AddJarCommand(path: String) extends RunnableCommand {
3838

3939
override def run(sparkSession: SparkSession): Seq[Row] = {
4040
sparkSession.sessionState.resourceLoader.addJar(path)
41-
Seq(Row(0))
41+
Seq.empty[Row]
4242
}
4343
}
4444

0 commit comments

Comments
 (0)