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

    https://github.com/apache/carbondata/pull/2791#discussion_r222409727
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonFileMetastore.scala
 ---
    @@ -208,7 +209,10 @@ class CarbonFileMetastore extends CarbonMetaStore {
         try {
           lookupRelation(tableIdentifier)(sparkSession)
         } catch {
    -      case _: Exception =>
    +      case ex: Exception =>
    +        if (ex.getCause.isInstanceOf[HiveException]) {
    +          throw ex
    +        }
    --- End diff --
    
    i checked the flow and permission errors will be thrown by Hive as 
HiveException and wrapped as analysis exception, and all the other exceptions 
will be same as i have described in description, during that time we cannot 
just catch exception and return false as table not exists. so i m checking as 
HiveException


---

Reply via email to