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

    https://github.com/apache/spark/pull/21655#discussion_r198748442
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
 ---
    @@ -1366,4 +1367,18 @@ class SessionCatalog(
         // copy over temporary views
         tempViews.foreach(kv => target.tempViews.put(kv._1, kv._2))
       }
    +
    +  private def validateLocationOfRename(oldName: TableIdentifier, newName: 
TableIdentifier): Unit = {
    --- End diff --
    
    Can you add code comments along with `validateName`?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to