yep, unpersist also works fine. I have made all the relevant changes as per
the code review. removed the option from the unpersist method as well to
make it more readable

Ram


On Wed, Jul 10, 2013 at 12:14 AM, Reynold Xin <notifications@github.com>wrote:

> In src/main/scala/shark/parse/SharkDDLSemanticAnalyzer.scala:
>
> > @@ -0,0 +1,23 @@
> > +package shark.parse
> > +
> > +import org.apache.hadoop.hive.conf.HiveConf
> > +import org.apache.hadoop.hive.ql.parse.{ASTNode, BaseSemanticAnalyzer, DDLSemanticAnalyzer, HiveParser}
> > +import shark.{LogHelper, SharkEnv}
> > +import spark.RDD
> > +import spark.rdd.UnionRDD
> > +
> > +class SharkDDLSemanticAnalyzer(conf: HiveConf) extends DDLSemanticAnalyzer(conf) with LogHelper {
> > +
> > +  override def analyzeInternal(node: ASTNode): Unit = {
> > +    super.analyzeInternal(node)
> > +    //handle drop table query
> > +    if (node.getToken().getType() == HiveParser.TOK_DROPTABLE) {
> > +      SharkEnv.unpersist(getTableName(node))
>
> I was referring to the unpersist method
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/amplab/shark/pull/104/files#r5109098>
> .
>