Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2704#discussion_r217021511
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/executor/util/RestructureUtil.java
---
@@ -165,14 +165,15 @@ private static boolean isColumnMatches(boolean
isTransactionalTable,
// column ID but can have same column name
if (tableColumn.getDataType().isComplexType() &&
!(tableColumn.getDataType().getId()
== DataTypes.ARRAY_TYPE_ID)) {
- if (tableColumn.getColumnId().equals(queryColumn.getColumnId())) {
+ if
(tableColumn.getColumnId().equalsIgnoreCase(queryColumn.getColumnId())) {
return true;
} else {
return isColumnMatchesStruct(tableColumn, queryColumn);
}
} else {
- return (tableColumn.getColumnId().equals(queryColumn.getColumnId())
|| (!isTransactionalTable
- && tableColumn.getColName().equals(queryColumn.getColName())));
+ return
(tableColumn.getColumnId().equalsIgnoreCase(queryColumn.getColumnId()) || (
--- End diff --
There would not be restructure for non transactional table. And dso the
case sensitivity check only for safety.
---
