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

    https://github.com/apache/carbondata/pull/2791#discussion_r221447314
  
    --- Diff: 
integration/spark2/src/main/commonTo2.2And2.3/org/apache/spark/sql/hive/CarbonInMemorySessionState.scala
 ---
    @@ -157,7 +158,13 @@ class InMemorySessionCatalog(
       }
     
       override def lookupRelation(name: TableIdentifier): LogicalPlan = {
    -    val rtnRelation = super.lookupRelation(name)
    +    val rtnRelation: LogicalPlan =
    +      try {
    --- End diff --
    
    why we need this try/catch, its super class doesn't throw any exception 
right, moreover we are throwing out the same exception object, can you please 
define the intention of this statement?


---

Reply via email to