Github user viirya commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19772#discussion_r151813095
  
    --- Diff: 
mllib/src/main/scala/org/apache/spark/ml/feature/SQLTransformer.scala ---
    @@ -70,7 +70,8 @@ class SQLTransformer @Since("1.6.0") (@Since("1.6.0") 
override val uid: String)
         dataset.createOrReplaceTempView(tableName)
         val realStatement = $(statement).replace(tableIdentifier, tableName)
         val result = dataset.sparkSession.sql(realStatement)
    -    dataset.sparkSession.catalog.dropTempView(tableName)
    --- End diff --
    
    Yes. I also think it doesn't make a lot sense.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to