Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2297#discussion_r187551542
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchemaCommon.scala
---
@@ -393,13 +393,6 @@ class TableNewProcessor(cm: TableModel) {
var allColumns: Seq[ColumnSchema] = Seq[ColumnSchema]()
fieldChildren.foreach(fields => {
fields.foreach(field => {
- if (!useDictionaryEncoding &&
--- End diff --
Don't remove this check in normal flow.
---