Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1525#discussion_r151831251 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/management/AlterTableCompactionCommand.scala --- @@ -87,8 +88,10 @@ case class AlterTableCompactionCommand( } catch { case e: Exception => if (null != e.getMessage) { + // TODO: We should modify it when we support compaction later. sys.error(s"Compaction failed. Please check logs for more info. ${ e.getMessage }") --- End diff -- Why not change this also?
---