Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1605#discussion_r154965672
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/management/CarbonAlterTableCompactionCommand.scala
---
@@ -159,7 +161,14 @@ case class CarbonAlterTableCompactionCommand(
// if system level compaction is enabled then only one compaction can
run in the system
// if any other request comes at this time then it will create a
compaction request file.
// so that this will be taken up by the compaction process which is
executing.
- if (!isConcurrentCompactionAllowed) {
+ if (carbonTable.isChildDataMap) {
--- End diff --
please modify the comment start from line 161, put them in correct place
and add comment for this newly added if condition
---
