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

    https://github.com/apache/carbondata/pull/1951#discussion_r166845879
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/package.scala
 ---
    @@ -87,8 +87,8 @@ abstract class AtomicRunnableCommand
       extends RunnableCommand with MetadataProcessOpeation with 
DataProcessOperation {
     
       override def run(sparkSession: SparkSession): Seq[Row] = {
    -    processMetadata(sparkSession)
    --- End diff --
    
    This change may impact other commands. Better don't include in try block. 
Let the individual command handle it.


---

Reply via email to