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

    https://github.com/apache/spark/pull/1038#discussion_r13619734
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala
 ---
    @@ -130,6 +130,11 @@ case class ExplainCommand(plan: LogicalPlan) extends 
Command {
     }
     
     /**
    + * Returned for the "CACHE TABLE tableName" and "UNCACHE TABLE tableName" 
command.
    + */
    +case class CacheCommand(tableName: String, doCache: Boolean) extends 
Command
    --- End diff --
    
    You can do this in the next PR if you prefer, but can we move all of these 
commands to their own file?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to